-
-
Notifications
You must be signed in to change notification settings - Fork 402
feat: build windows distributions with cibuildwheel in GitHub CI #1423
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
Should avoid maintaining appveyor workflow (and free up appveyor account limitations 😉). This also introduces windows aarch64 distributions 🎉 Lastly, I updated other spots in the GH actions workflow: - upgrade `actions/checkout` to v5 (only breaking changes are internal like upgrade node version used for building their dist) - turn off credential persistence in `actions/checkout` action (only useful if the workflow uses `git push` or similar) - upgrade `actions/download-artifact` to v5 (only breaking changes are internal because this already uses `merge-multiple: true`) - add job to check built distributions for non-tagged commits to default branch. This is meant to highlight any problems with dists' metadata before tagging a release. - add job to build sdist (for sake of completeness) - update README.md badges
a92394b to
098cbe3
Compare
|
I never used appveyor before. I was assuming that if the appveyor.yml was removed then, appveyor may throw some kind of error. Whatever works for you is good enough for me. |
I do have some experience here. If you decide to use trusted publishing (instead of using a long-lived API token), I would recommend leaving the "environment" field blank. It only complicates the needed CI workflow changes. Please ping me if/when you are ready to adopt trusted publishing. |
This idea is not really necessary, but it would notify people (that are watching or have starred this repo) about new releases. Additionally, this idea can be used to generate a summary of commits since the last tag. However, there are other ways to auto-generate changelog entries. Personally, I enjoy git-cliff, but it works best with conventional-commit messages (or GitHub PR labels). |
|
you were right, it produced an error in appveyor; anyway, I have removed both appveyor in pygit2 and the pygit2 project in appveyor, thanks! |
|
I'm playing around more with cibuildwheel. Trying to get it to run tests for each build (even the ones obsoleted by abi3 builds). So far, there is no problems if the following snippet is commented out: Lines 25 to 28 in e5620b9
Although I am getting the |
This should avoid maintaining appveyor workflow (and free up appveyor account limitations 😉).
This also introduces windows aarch64 distributions 🎉
Lastly, I updated other spots in the GH actions workflow:
actions/checkoutto v5 (only breaking changes are internal like upgrade node version used for building their dist)actions/checkoutaction (only useful if the workflow usesgit pushor similar)actions/download-artifactto v5 (only breaking changes are internal because this already usesmerge-multiple: true)Further Ideas
Ideas that are related to this.
These can be added to this PR with ease:
These should be done in a separate PR: