ci: drop the macos-13 (Intel macOS) wheel leg - #78
Merged
Conversation
The macos-13 hosted runner does not schedule for this repo, so the `wheel x86_64 (macos-13)` job hangs `queued` indefinitely. `build-wheels` is a matrix job that `publish` depends on, so that one stuck leg blocks the entire PyPI release — the publish job is never reachable. The first release-python dispatch stalled on exactly this. Drop the Intel-macOS leg so the matrix completes on its four available runners (linux x86_64 + aarch64, macOS arm64, windows x64). Intel-mac users install from the sdist (it builds the extension from source); a cross-compiled Intel wheel is tracked in #29. Signed-off-by: Nelson Spence <nelson@projectnavi.ai>
Nelson Spence (Fieldnote-Echo)
requested a review
from Navi Bot (project-navi-bot)
as a code owner
May 26, 2026 17:13
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
Review Summary by QodoRemove macos-13 Intel macOS wheel from release CI matrix
WalkthroughsDescription• Remove macos-13 Intel wheel from CI matrix • Unblocks PyPI release by eliminating queued job • Adds explanatory comment about Intel wheel strategy • Maintains four available runner configurations Diagramflowchart LR
A["release-python.yml"] --> B["build-wheels matrix"]
B --> C["macos-13 x86_64 removed"]
C --> D["4 runners remain active"]
D --> E["publish job unblocked"]
File Changes1. .github/workflows/release-python.yml
|
Code Review by Qodo
Context used 1. Stale Intel wheel comment
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
project-navi-bot
approved these changes
May 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Unblocks the PyPI release of
ordvec0.2.0.Problem
The
release-python.ymlwheel matrix includes amacos-13(Intel macOS) leg, but macos-13 hosted runners don't schedule for this repo — the job hangsqueuedindefinitely.build-wheelsis a matrix job thatpublishdepends on, so that one stuck leg blocks the whole PyPI release (the publish job is never reachable). The firstrelease-pythondispatch stalled on exactly this.Fix
Drop the
macos-13leg. The matrix completes on its four available runners (linux x86_64 + aarch64, macOS arm64, windows x64). Intel-mac users install from the sdist (which compiles the extension from source); a cross-compiled Intel wheel is tracked in #29.Context
ordvec0.2.0) is already live on crates.io — that leg published fine.mainHEAD goes green, re-dispatchrelease-python.ymlto publish the wheel to PyPI, then approve thepypienvironment.