-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
remove HOMEBREW_MAKE #11196
remove HOMEBREW_MAKE #11196
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.
Nice work again @hyuraku!
Head branch was pushed to by a user without write access
65df4a2
to
b9202d9
Compare
b9202d9
to
40084fc
Compare
# shellcheck disable=SC2154 | ||
# HOMEBREW_CCCFG is set by extend/ENV/super.rb |
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.
This comment seems to refer to HOMEBREW_CCCFG
instead of HOMEBREW_MAKE
. Is it correct for it to be removed?
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.
Same question! As long as shellcheck
passes I'm fine with it being removed if it's no longer needed.
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.
I removed the comments because brew style
with shellcheck
passed without them.
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.
Cool, just checking! 👍
Thanks again @hyuraku! |
brew style
with your changes locally?brew typecheck
with your changes locally?brew tests
with your changes locally?remove
HOMEBREW_MAKE
block because it was undocumented.related #11179 (comment)