Skip to content
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

Add python 3.12 wheels on v3.7 #344

Merged
merged 6 commits into from
Mar 9, 2024
Merged

Add python 3.12 wheels on v3.7 #344

merged 6 commits into from
Mar 9, 2024

Conversation

msaecker
Copy link

@msaecker msaecker commented Feb 26, 2024

Extend the exisiting build pipeline to create python 3.12 wheels.

@CLAassistant
Copy link

CLAassistant commented Feb 26, 2024

CLA assistant check
All committers have signed the CLA.

@msaecker msaecker marked this pull request as ready for review February 26, 2024 10:37
Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (f8958ac) to head (8f88192).

Additional details and impacted files
@@            Coverage Diff            @@
##                v3      #344   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           17        17           
  Lines          425       425           
=========================================
  Hits           425       425           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@isaacbrodsky
Copy link
Collaborator

This does not seem to be building correctly? https://github.com/uber/h3-py/actions/runs/8047484474/job/22005886719?pr=344

@msaecker
Copy link
Author

This does not seem to be building correctly? https://github.com/uber/h3-py/actions/runs/8047484474/job/22005886719?pr=344

Yeah, the cibuildwheel version did not support python 3.12 yet. I upgraded to the newest one. The earliest stable ABI was in version 2.15, with the final CPython 3.12 in v2.16.2.

ajfriend and others added 3 commits February 29, 2024 07:51
* Drop cibuildwheel v1 and 3.5 tests

* try [email protected]

* drop down to [email protected]

* bump ubuntu from 20.04 to 22.04

* Revert "bump ubuntu from 20.04 to 22.04"

This reverts commit f94bea6.

* bump Python version from 3.8 to 3.11 for coverage/lint and docs

* bump cibuildwheel
@msaecker
Copy link
Author

I cherry-picked some commits from main dropping c1 builds and python 2.7 & 3.5. I also added a commit dropping 3.6.

@msaecker
Copy link
Author

msaecker commented Mar 1, 2024

The cherry-picking seems to have overwritten the cibuildversion to 2.16.2 instead of 2.16.5, but that should be fine as it contains a final CPython version for 3.12. I have no idea why the windows build failed and this is as far as I am taking this PR.

@msaecker
Copy link
Author

msaecker commented Mar 4, 2024

Upgraded cibuildwheel back to 2.16.5 as this might be the cause of the windows build issues.

@msaecker
Copy link
Author

msaecker commented Mar 6, 2024

All green.

@dfellis dfellis requested a review from ajfriend March 6, 2024 15:06
@ajfriend
Copy link
Contributor

ajfriend commented Mar 9, 2024

This looks good. Thanks @msaecker!

@ajfriend ajfriend merged commit b04ac8b into uber:v3 Mar 9, 2024
36 checks passed
@msaecker msaecker deleted the v3-python-3.12-wheels branch March 11, 2024 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants