-
Notifications
You must be signed in to change notification settings - Fork 80
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
[WIP] Add windows and mac-arm64 wheels #1334
Conversation
Codecov Report
@@ Coverage Diff @@
## latest #1334 +/- ##
==========================================
- Coverage 81.04% 80.98% -0.07%
==========================================
Files 102 104 +2
Lines 10298 10312 +14
Branches 1165 1165
==========================================
+ Hits 8346 8351 +5
- Misses 1744 1753 +9
Partials 208 208
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
dd7a917
to
cf55675
Compare
#1115 might fix the issue with windows builds |
cf55675
to
4d050d4
Compare
I'll probably split this PR into two: one for windows wheels, another for |
Previous try (in #1334) tried to build `universal2` wheels on mac, but that's complicated. This PR builds two separated wheels, one for `arm64`, another for `x86_64`, and let `pip` choose the correct one during installation. I also updated `cibuildwheel` to list configuration in `pyproject.toml`, this way it can also be tested locally. (In preparation for fixing the currently broken `sourmash-minimal` package on `conda-forge`, which is not doing the cross-compilation properly).
(This is built on top of #1564 now)
Checklist
make test
Did it pass the tests?make coverage
Is the new code covered?without a major version increment. Changing file formats also requires a
major version number increment.
changes were made?