-
-
Notifications
You must be signed in to change notification settings - Fork 9.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
Make Sequoia supported #18296
Make Sequoia supported #18296
Conversation
a0485d0
to
5893597
Compare
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.
Makes sense! As mentioned: we should aim to merge this as soon as we've got Sequoia CI running on homebrew/core PRs. Anything that needs to wait longer: other folks please suggest changes so we can scope this down to something to merge this week.
…orted_release?` See discussion at #18296.
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.
#18306 will decouple these values from what's running in Homebrew/core CI, so this should be good to go any time after that for me.
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 concerns resolved for me too. This can safely sit in master
now.
Note that the Homebrew/install PR should wait until the next tag.
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 @fxcoudert for opening and @carlocab and @Bo98 for help getting this over the line!
5893597
to
4ab19e0
Compare
@carlocab Some CI failures here related to GitHub runners; could you take a look? Thanks 🙇🏻 |
Yup, tests just need updating since the tests assume they follow |
We've decoupled the macOS versions there from `MacOSVersion#supported_release?`, so these tests need updating.
Pushed an update here. Not complete yet, but will get there soon. |
Great, thanks for this @carlocab! |
Huh, didn't expect tests to pass yet. Glad that's all that was needed. 🥳 Will fix up those test assumptions at some point. |
macOS Sequoia RC has dropped, and release is near. This PR is open to discuss the timing (we need to coordinate the merge of this PR and make a new release soon/immediately after).
Companion PR: Homebrew/install#878