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
Homebrew::EnvConfig: unify environment variables. #7285
Conversation
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.
Thanks for the review @Bo98, nice catches!
f4bf3ab
to
92d98d7
Compare
59f5808
to
31ed7ca
Compare
@Bo98 Ready for re-review, if you're game! Still have some unit tests to write but otherwise think this is good to go. |
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.
LGTM
cli/parser but for environment variables. This makes it easier to set, check, document and set defaults for environment variables in a single location that can be used for query and documentation generation.
31ed7ca
to
f8c01c4
Compare
f8c01c4
to
b2895e1
Compare
b2895e1
to
6dbc611
Compare
6dbc611
to
3381613
Compare
Like
cli/parser
but for environment variables. This makes it easier to set, check, document and set defaults for environment variables in a single location that can be used for query and documentation generation.TODO: write unit tests for
Homebrew::EnvConfig
if no-one objects to the approach.brew style
with your changes locally?brew tests
with your changes locally?