Skip to content

CI: Don't run macos (x86) in full#39968

Merged
jschwe merged 1 commit intoservo:mainfrom
jschwe:jschwender/ci-macos
Oct 18, 2025
Merged

CI: Don't run macos (x86) in full#39968
jschwe merged 1 commit intoservo:mainfrom
jschwe:jschwender/ci-macos

Conversation

@jschwe
Copy link
Member

@jschwe jschwe commented Oct 17, 2025

Only do x86 builds on macos on push and on nightly builds. Also test building libservo on arm mac instead of x86 mac. Also fixes an issue when merging macos arm jobs in the try parser.
The main goal of this PR is to improve CI merge times again (since the concurrency limit of github-hosted runners, has been slowing down the MQ a lot).

Testing: This changes a CI workflow.

Only do x86 builds on macos on push and on nightly builds.
Also test building libservo on arm mac instead of x86 mac.
Also fixes an issue when merging macos arm jobs in the try parser.
The main goal of this PR is to improve CI merge times again.

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Oct 17, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Oct 17, 2025
@jschwe jschwe enabled auto-merge October 17, 2025 19:25
@jschwe jschwe added this pull request to the merge queue Oct 17, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 17, 2025
@yezhizhen
Copy link
Member

This should probably be the first priority to be merged given that we have 11 in the queue rn.Screenshot_2025_1018_045821.png

And those PR's macOS intel UT all have "waiting for runner to pick up" while other tasks have almost finished.

Merged via the queue into servo:main with commit f8ffa13 Oct 18, 2025
33 checks passed
@jschwe jschwe deleted the jschwender/ci-macos branch October 18, 2025 03:02
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Oct 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants