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
Improve homebrew/{core,cask} autotapping. #14613
Conversation
Review period skipped due to |
28e376f
to
1568474
Compare
Library/Homebrew/brew.sh
Outdated
@@ -805,6 +805,7 @@ then | |||
if check-array-membership "${HOMEBREW_COMMAND}" "${NO_INSTALL_FROM_API_COMMANDS[@]}" | |||
then | |||
export HOMEBREW_NO_INSTALL_FROM_API=1 | |||
export HOMEBREW_NO_INSTALL_FROM_API_SET_AUTOMATICALLY=1 |
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.
Should we set this only when HOMEBREW_NO_INSTALL_FROM_API
is not already set by the user?
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.
@carlocab Yes, good catch.
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.
Looks great, thanks!
Will the dev commands still work correctly after this without tapping the core taps automatically? Or will they break like before? |
It depends what you mean by "work correctly". If they aren't tapped and you try to |
1568474
to
b549441
Compare
Don't automatically tap these when running a developer command that's not using the API. Fixes #14606
b549441
to
4c8ed77
Compare
Don't automatically tap these when running a developer command that's not using the API.
Fixes #14606