Skip to content

add config validation by default #844

Description

@shazron

Right now, config validation is not turned on by default and only certain commands turn it on, like the app pack command:

const appConfig = await libConfig.load({ validateAppConfig: true })

Modify BaseCommand.getFullConfig to validate the app config by default - this means that all commands that call getFullConfig will validate the config by default, even if they don't use the whole config:
https://github.com/search?q=repo%3Aadobe%2Faio-cli-plugin-app%20getFullConfig&type=code

BaseCommand.getFullConfig:

const validateAppConfig = options.validateAppConfig === true

Provide a shared flag in the BaseCommand to turn off config validation, just in case there are issues:

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions