Skip to content

Fix wheel builds on CI #20153

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

Closed
wants to merge 5 commits into from
Closed

Fix wheel builds on CI #20153

wants to merge 5 commits into from

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented May 4, 2021

PR Summary

The minimum NumPy with Python 3.8 wheels is 1.17.3, so bump CI to use that version. This should be fine since there aren't supposed to be ABI changes in micro releases.

Also drop remnants of 3.6 builds, that are mostly disabled already.

Corresponds to this test build, which is working again

PR Checklist

  • [n/a] Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (run flake8 on changed files to check).
  • [n/a] New features are documented, with examples if plot related.
  • [n/a] Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • Conforms to Matplotlib style conventions (install flake8-docstrings and run flake8 --docstring-convention=all).
  • [n/a] New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • [n/a] API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).

QuLogic added 2 commits May 3, 2021 23:46
This was only enabled on the v3.3.x branch, so can be dropped from 3.5,
and even 3.4 (but not going to bother).
This is the minimum available for Python 3.8 in wheel form, and should
not break ABI to use.
@QuLogic QuLogic added this to the v3.5.0 milestone May 4, 2021
@QuLogic
Copy link
Member Author

QuLogic commented May 4, 2021

Oops, this is from the test branch.

@QuLogic
Copy link
Member Author

QuLogic commented May 4, 2021

Correct branch is at #20155.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant