-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
setup-homebrew: update tweaks. #393
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.
I think we need to scope Homebrew/core out of this behaviour currently, because we do want full clones there.
Should already be I think. homebrew-core in general is a special case in this script. Except the test-bot bit but I've commented on that above. |
63a0af2
to
3edce1b
Compare
- `HOMEBREW_NO_INSTALL_FROM_API` was passed in a few places unnecessarily - Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
7b06819
to
2bd38cf
Compare
2bd38cf
to
7fdc0b2
Compare
See discussion at Homebrew/actions#393.
|
See discussion at Homebrew/actions#393.
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.
All 🟢
- `HOMEBREW_NO_INSTALL_FROM_API` was passed in a few places unnecessarily - Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
7fdc0b2
to
9f9f684
Compare
- don't update core/cask/test-bot by default - unset HOMEBREW_NO_INSTALL_FROM_API (which GitHub Actions sets by default - use `brew update` to download formula/cask files Co-authored-by: Bo Anderson <mail@boanderson.me>
9f9f684
to
f9eb51b
Compare
- `HOMEBREW_NO_INSTALL_FROM_API` was passed in a few places unnecessarily - Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
- `HOMEBREW_NO_INSTALL_FROM_API` was passed in a few places unnecessarily - Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
- `HOMEBREW_NO_INSTALL_FROM_API` was passed in a few places unnecessarily - Homebrew/actions#393 will change the default behaviour of setup-homebrew so let's be explicit to speed things up and avoid changes when it's merged
brew update
to download formula/cask filesIf this is ✅: we'll need to audit our usage of this action to ensure they are configured correctly to allow this change but it should reduce load on GitHub, failures seen by us and our users and better test the default Homebrew setup.