Skip to content

Remove macOS Intel CI pipeline#1991

Merged
nicolasnoble merged 2 commits intogrumpycoders:mainfrom
nicolasnoble:remove-macos-intel-ci
Mar 24, 2026
Merged

Remove macOS Intel CI pipeline#1991
nicolasnoble merged 2 commits intogrumpycoders:mainfrom
nicolasnoble:remove-macos-intel-ci

Conversation

@nicolasnoble
Copy link
Member

@nicolasnoble nicolasnoble commented Mar 24, 2026

GitHub has deprecated the macos-13 runner, which was the last
free-tier Intel macOS runner. Remove the Intel build and publish
jobs from the macOS CI workflow, and update the README to point
to the ARM build only.

GitHub has deprecated the macos-13 runner, which was the last
free-tier Intel macOS runner. Remove the Intel build and publish
jobs from the macOS CI workflow, and update the README to point
to the ARM build only.

Signed-off-by: Nicolas 'Pixel' Noble <nicolas@nobis-crew.org>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 470645d8-8877-4d35-aeb9-1fdb6cbc9119

📥 Commits

Reviewing files that changed from the base of the PR and between 4152915 and ad61632.

📒 Files selected for processing (1)
  • README.md
✅ Files skipped from review due to trivial changes (1)
  • README.md

📝 Walkthrough

Walkthrough

Removed Intel (x64) macOS build and publish jobs from CI and updated documentation to reference Arm-only macOS builds and downloads.

Changes

Cohort / File(s) Summary
CI/CD Workflow Simplification
.github/workflows/macos-build.yml
Deleted the macos-intel-build-and-test job and its dependent publish-intel-app job; Intel DMG bundling, signing, packaging, and artifact uploads removed.
Documentation Updates
README.md
Removed Intel/macOS x64 entries and download links; replaced badge and download references to point to the Arm-only macOS build.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

size/M

Poem

🐰 I hopped through CI with a cheerful twitch,
Intel footprints vanished—Arm found its niche.
Docs trimmed tidy, the builds now lean,
A carrot of clarity on the project scene. 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: removing the macOS Intel CI pipeline from the workflow and README, which is the core purpose of this PR.
Description check ✅ Passed The description clearly explains why the changes are being made (GitHub deprecated macos-13 runner) and what is being changed (Intel build/publish jobs and README updates).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 9: Update the capitalization of Apple platform labels in the markdown
table: replace the "MacOS build" cell label with "macOS build" and the link text
"[MacOS Arm]" with "macOS Arm" so they match Apple's official "macOS" casing and
the rest of the README; make sure to update both the visible table cell text and
the bracketed link text exactly where "MacOS build" and "MacOS Arm" appear.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9c58f5ee-4143-4a51-b9e8-f4b6d238effa

📥 Commits

Reviewing files that changed from the base of the PR and between 45fcccd and 4152915.

📒 Files selected for processing (2)
  • .github/workflows/macos-build.yml
  • README.md
💤 Files with no reviewable changes (1)
  • .github/workflows/macos-build.yml

@codecov
Copy link

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 9.11%. Comparing base (acdc7e5) to head (4152915).
⚠️ Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1991      +/-   ##
==========================================
- Coverage    9.30%    9.11%   -0.20%     
==========================================
  Files         469      470       +1     
  Lines      145164   147911    +2747     
==========================================
- Hits        13513    13482      -31     
- Misses     131651   134429    +2778     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Nicolas 'Pixel' Noble <nicolas@nobis-crew.org>
@nicolasnoble nicolasnoble enabled auto-merge March 24, 2026 01:36
@nicolasnoble nicolasnoble disabled auto-merge March 24, 2026 02:13
@nicolasnoble nicolasnoble merged commit 593f94c into grumpycoders:main Mar 24, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant