Fix MacOS Intel runners #1
Conversation
cibuildwheel.yml
Outdated
| os: | ||
| - ubuntu-latest # Intel | ||
| - ubuntu-24.04-arm # ARM | ||
| - windows-latest # Intel | ||
| - windows-11-arm # ARM | ||
| - macos-15-intel # Intel | ||
| - macos-latest # ARM |
There was a problem hiding this comment.
This example file looks severely obsolete
There was a problem hiding this comment.
Going by the commit log, I think this file and the publish_pypy.yml files were accidentally left here when the "real" files were moved to .github/workflows.
I'd just delete both of them.
There was a problem hiding this comment.
publish_pypy was never moved though.
A proper cleanup looks very messy and beyond the scope of this PR.
There was a problem hiding this comment.
Maybe just delete this file? Either way it's not worth updating like you have it here.
There was a problem hiding this comment.
I've just reverted the changes in this PR to keep a clean git history.
README.md
Outdated
| uses: explosion/gha-cibuildwheel/cibuildwheel.yml@main | ||
| with: | ||
| os-matrix: '["ubuntu-latest", "windows-latest", "macos-13", "macos-14", "ubuntu-24.04-arm"]' | ||
| os-matrix: '["ubuntu-latest", "ubuntu-24.04-arm", "windows-latest", "windows-11-arm", "macos-15-intel", "macos-latest"]' |
There was a problem hiding this comment.
This looks obsolete.
I found no trace of this os-matrix setting. It's hardcoded in .github/workflows/cibuildwheel.yml:113.
There was a problem hiding this comment.
It looks like the docs were added in 1b1ad35, which has the message "Add LLM docs" so it was probably hallucinated.
2 participants
macos-13runners have been retired and replaced bymacos-15-intel.