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 3.13 support, remove 3.8 #202

Merged
merged 3 commits into from
Nov 8, 2024
Merged

Conversation

ChrisLovering
Copy link
Contributor

CI will fail on 3.13 builds until actions/python-versions#312 is merged to add 3.13 support to setup-python

@@ -55,7 +55,7 @@ jobs:
run: echo "CIBW_ARCHS=${{ matrix.cibw_archs }}" >> $GITHUB_ENV
shell: bash
- name: Build wheels
uses: pypa/cibuildwheel@v2.16.2
uses: pypa/cibuildwheel@v2.21.2
Copy link
Contributor Author

@ChrisLovering ChrisLovering Oct 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As of writing this version is both the latest, and the one that adds 3.13 support
https://github.com/pypa/cibuildwheel/releases

@ChrisLovering ChrisLovering marked this pull request as draft October 8, 2024 10:30
@ChrisLovering ChrisLovering marked this pull request as ready for review October 8, 2024 15:37
@ChrisLovering
Copy link
Contributor Author

ChrisLovering commented Oct 8, 2024

actions/python-versions#312 has been merged, and 3.13 CI has passed here. Unsure why the macOS runners aren't picking up the outstanding jobs though.

@saghul this is ready for review :)

@saghul
Copy link
Owner

saghul commented Oct 8, 2024

This is awesome, thank you!

@ChrisLovering
Copy link
Contributor Author

macOS CI failed as no runner picked up the jobs within a day, rather than due to an error in the change itself.

That doesn't mean there isn't a problem with these changes, just that we don't know yet.

@saghul
Copy link
Owner

saghul commented Oct 14, 2024

macOS 11 was decomissioned, so we need to update

os: [ubuntu-20.04, windows-2019, macos-11]

@ChrisLovering
Copy link
Contributor Author

Updated CI to run using macOS 13 (newest x86) and macOS 15 for M1 support.

Various mac versions are failing the test_custom_resolvconf test due to the errorno not being the expected value. Unsure on the cause of this, as some version of Python finish tests just fine, and the ones that fail aren't consistent across macOS 13 & 15.

@candrews
Copy link

Can we get this merged and released?

I'm really looking forward to Python 3.13 support :)

Thank you!

@saghul
Copy link
Owner

saghul commented Oct 22, 2024

Tests are failing, any help with that would be welcome!

@ChrisLovering
Copy link
Contributor Author

Yeah, it looks like it's a potential flaky test, as it works on some Python versions on macOS 13 and then other Python versions on macOS 15.

I don't have a mac so can't reproduce in order to fix it.

@Zaczero
Copy link

Zaczero commented Nov 5, 2024

It doesn't look like the failing tests are related to the Python 3.13 upgrade.

@No767
Copy link

No767 commented Nov 8, 2024

I've been looking into the tests and my best guess is that it has to do with something with MacOS or how the tests are set up

@@ -9,8 +9,8 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-20.04, windows-2019, macos-11]
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
os: [ubuntu-20.04, windows-2019, macos-13, macos-15]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something of note is that we should be testing against Windows 2022 and Ubuntu 24.04 if we are upgrading macos versions as well

@saghul saghul merged commit da178fd into saghul:master Nov 8, 2024
22 of 26 checks passed
@ChrisLovering ChrisLovering deleted the drop-3.8-add-3.13 branch November 10, 2024 18:02
@monosans
Copy link
Contributor

Could you please upload CPython 3.13 wheels to PyPI?

@ChrisLovering
Copy link
Contributor Author

hey @saghul is there anything blocking making a release to pypi for this PR that we can help with?

@saghul
Copy link
Owner

saghul commented Nov 23, 2024

I tried to publish them but ran into a problem. Looks like a v3 / v4 upload action shenanigans.

I'll try to look into it soon.

@monosans
Copy link
Contributor

Hey, @saghul. I've created a PR which should fix the CI release and also added more platforms to cibuildwheel. See #204.

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.

6 participants