Skip to content

ci(python-wheels): cross-compile Intel-mac wheel on Apple-silicon runner - #1319

Merged
louistrue merged 1 commit into
mainfrom
ci/python-wheels-macos-cross
Jun 22, 2026
Merged

ci(python-wheels): cross-compile Intel-mac wheel on Apple-silicon runner#1319
louistrue merged 1 commit into
mainfrom
ci/python-wheels-macos-cross

Conversation

@louistrue

@louistrue louistrue commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

The macos-13 (Intel) runners are being deprecated and queue indefinitely, hard-blocking the ifclite-geom PyPI publish job (it needs all 5 wheels). Build the x86_64-apple-darwin wheel on the macos-14 (Apple-silicon) runner via maturin cross-compile instead; abi3 links no Python lib so the cross target builds cleanly. No Rust code change — wheel contents are identical to main.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated macOS build infrastructure to maintain compatibility with the latest supported runner versions.

The macos-13 (Intel) runners are being deprecated and queue indefinitely,
hard-blocking the PyPI publish job (which needs all 5 wheels). Build the
x86_64-apple-darwin wheel on the macos-14 runner instead; abi3 links no
Python lib so the cross target builds cleanly.
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ifc-lite Ready Ready Preview, Comment Jun 22, 2026 4:33pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
ifc-lite-dev Ignored Ignored Jun 22, 2026 4:33pm
ifc-lite-viewer-embed Ignored Ignored Jun 22, 2026 4:33pm

@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6d9dc03e-c105-405e-95d8-34d781396124

📥 Commits

Reviewing files that changed from the base of the PR and between 8d3aeb4 and bef8c18.

📒 Files selected for processing (1)
  • .github/workflows/python-wheels.yml

📝 Walkthrough

Walkthrough

The GitHub Actions python-wheels.yml workflow is updated to replace the macos-13 runner with macos-14 for the Intel x86_64 macOS wheel build matrix entry. Inline comments are expanded to describe cross-compiling Intel-mac wheels on Apple-silicon runners and the abi3 wheel format rationale.

Changes

macOS Intel Wheel CI Runner Update

Layer / File(s) Summary
macOS x86_64 runner and matrix comments
.github/workflows/python-wheels.yml
The build matrix include entry for the Intel x86_64 macOS wheel is changed from macos-13 to macos-14, and inline comments are updated to explain the cross-compile approach and abi3 behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • LTplus-AG/ifc-lite#255: Updates the same workflow file to use macos-14 instead of macos-13 for the macOS Intel (darwin-x64/x86_64) build matrix, with matching cross-compile rationale.

Poem

🐇 A runner retired, macos-13 no more,
Now macos-14 compiles the x86 core.
Cross-compile magic on Apple-silicon ground,
The abi3 wheel spins merrily around.
Hoppy days for CI, steady and sound! 🍎

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the CI workflow to cross-compile Intel-mac wheels on Apple-silicon runners, replacing deprecated macos-13 with macos-14.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/python-wheels-macos-cross

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.

@louistrue
louistrue merged commit 9df939a into main Jun 22, 2026
26 checks passed
@louistrue
louistrue deleted the ci/python-wheels-macos-cross branch June 22, 2026 16:41
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.

1 participant