-
-
Notifications
You must be signed in to change notification settings - Fork 13
Add arm64 windows wheel builds #133
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
@finnagin Looks good. More wheels are definitely needed. I recently had someone add these to winloop recently too. If I was able to review this code (which I don't know if I have permission to do so) I would've done so. |
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
Adds workaround until actionlint includes windows-11-arm in list of valid `runs-on` options
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
Co-authored-by: 🇺🇦 Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
To explain that python versions are skipped here because setup-python only supports 3.11+ on arm64 windows
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.
Looks good I have zero complaints I am a bit newer over here so I am also trying to practice. I see nothing wrong with adding this in
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.
I have ran pre-commit locally since I don't have the power to force this workflow to run but this should pass now.
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.
LGTM, thanks!
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 pull request adds support for building ARM64 wheels on Windows by updating the CI/CD pipeline to include windows-11-arm
as a runner option. The changes enable native wheel distribution for ARM64 Windows users.
- Added
windows-11-arm
runner to the build matrix for both wheel building and testing workflows - Modified the reusable workflow to support OS values without the
-latest
suffix - Added actionlint configuration to recognize the new runner type
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
CHANGES/133.contrib.rst | Documents the addition of ARM64 Windows wheel builds |
.github/workflows/reusable-build-wheel.yml | Removes automatic -latest suffix to support windows-11-arm runner |
.github/workflows/ci-cd.yml | Adds windows-11-arm to build and test matrices with appropriate exclusions |
.github/actionlint.yml | Configures actionlint to recognize windows-11-arm as a valid runner |
Plz ignore copilot. I was just curious what it'd produce. |
What do these changes do?
This would add wheel builds for arm64 Windows.
Are there changes in behavior for the user?
No, other than a native wheel would be available for arm64 Windows users.
Related issue number
Fixes #132
Checklist