-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do not warn when env var matches argument alias (DAT-16117) #5165
Conversation
…les when the environment variable matches an alias.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there any tests for this code? Could you add to them or create one for this case?
Also, @wwillard7800 , didn't you make a ticket for this recently? I could be imagining things.
There's none for this particular case since it's integrating the configuration with the env var provider. But yeah, the test cases in this area are very light. I'll see what I can do. Edit: done! |
Kudos, SonarCloud Quality Gate passed! |
Impact
Description
Do not warn on unrecognized LIQUIBASE_ environment variables when the environment variable matches an alias.
Things to be aware of
Things to worry about
Additional Context