-
-
Notifications
You must be signed in to change notification settings - Fork 186
ci: switch to arm64 runners #829
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
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
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 updates GitHub Actions workflows to run CI jobs on Ubuntu 24.04 ARM runners to improve performance and reduce cost.
Changes:
- Switch
runs-onfromubuntu-latesttoubuntu-24.04-armin the main CI workflow. - Switch
runs-onfromubuntu-latesttoubuntu-24.04-armin the autofix workflow. - Switch
runs-onfromubuntu-latesttoubuntu-24.04-armin the Lunaria overview workflow.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/ci.yml | Moves lint/test/browser/a11y/knip jobs to Ubuntu 24.04 ARM runners. |
| .github/workflows/autofix.yml | Moves the autofix job to an Ubuntu 24.04 ARM runner. |
| .github/workflows/lunaria.yml | Moves Lunaria overview generation to an Ubuntu 24.04 ARM runner. |
📝 WalkthroughWalkthroughThis pull request updates three GitHub Actions workflows to change CI runner images. The autofix, CI and lunaria workflows have their 🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
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.
Actionable comments posted: 1
|
Actual difference in CI run time minuscule, a bit wobbly because of caching that didn't settle down just yet. Up to you whether to merge or not. |
|
There's also these new runners we could try for some of the lightweight workflows? https://github.blog/changelog/2026-01-22-1-vcpu-linux-runner-now-generally-available-in-github-actions/ |
Interesting, but it seems like a cost saving opportunity, which in our case is not a concern. Or is it? |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
From my experience, this gives ~20% performance boost, while being 17% cheaper.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
as a public repo, we're not paying for actions |
Comparing, (before/after; take into consideration missing cache) it seems to shave off:
leaving the rest pretty much intact.