-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
github/workflows: use macos-15 GitHub Actions runners. #18462
base: master
Are you sure you want to change the base?
Conversation
MikeMcQuaid
commented
Sep 30, 2024
- Let's use this instead of macOS 14/latest when possible.
- Keep around macOS 13 to ensure we're still testing x86_64.
- Let's use this instead of macOS 14/latest when possible. - Keep around macOS 13 to ensure we're still testing x86_64.
Blocked until GitHub sorts this out I guess. |
Pull request was converted to draft
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 suppose this will conflict with #18395 (which I forgot to merge). Can we that first if you don't mind?
@ZhongRuoyu Sure, done! |
@@ -84,7 +84,7 @@ def run | |||
test-bot: | |||
strategy: | |||
matrix: | |||
os: [ubuntu-22.04, macos-13, macos-14] | |||
os: [ubuntu-22.04, macos-13, macos-15] |
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.
Why not:
os: [ubuntu-22.04, macos-13, macos-15] | |
os: [ubuntu-22.04, macos-14, macos-15] |
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.
macOS 13 is x86_64. macOS 14 and 15 are arm64.