-
Notifications
You must be signed in to change notification settings - Fork 925
CI: run macos-15-intel only on main (not in PRs) #5926
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
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.
Pull request overview
This PR optimizes CI costs by configuring the macos-15-intel runner (macos-x64) to skip execution in pull requests and only run on the main branch or release branches. This is achieved by adding a run_in_pr flag to each build metadata configuration and implementing a conditional check at the job level.
Key changes:
- Added
run_in_prboolean field to all metadata entries in thebuildjob matrix - Set
run_in_pr: falseformacos-x64(macos-15-intel) to skip it in PRs - Set
run_in_pr: truefor all other platforms (linux-x64, macos-arm, windows-x64, windows-gnu, linux-musl) - Added job-level conditional to respect the
run_in_prflag and run all builds on non-PR events
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
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.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
d461e40 to
59147ba
Compare
dc1c6a9 to
6dc69d0
Compare
Depot does not provide macOS Intel runners, and so let's demote them as something we'll be testing only on
mainbranch (or a release branch).