Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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
@codecov
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
Collaborator

ajfriend commented Mar 9, 2024

This looks good. Thanks @msaecker!

@ajfriend ajfriend merged commit b04ac8b into uber:v3 Mar 9, 2024
@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