pycares icon indicating copy to clipboard operation
pycares copied to clipboard

Add 3.13 support, remove 3.8

Open ChrisLovering opened this issue 1 year ago • 8 comments

CI will fail on 3.13 builds until https://github.com/actions/python-versions/pull/312 is merged to add 3.13 support to setup-python

ChrisLovering avatar Oct 08 '24 10:10 ChrisLovering

https://github.com/actions/python-versions/pull/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 :)

ChrisLovering avatar Oct 08 '24 15:10 ChrisLovering

This is awesome, thank you!

saghul avatar Oct 08 '24 15:10 saghul

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.

ChrisLovering avatar Oct 14 '24 13:10 ChrisLovering

macOS 11 was decomissioned, so we need to update https://github.com/saghul/pycares/blob/57a82a0039ad2607092abe6aff0f06be17a400f2/.github/workflows/test.yml#L12

saghul avatar Oct 14 '24 13:10 saghul

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.

ChrisLovering avatar Oct 14 '24 15:10 ChrisLovering

Can we get this merged and released?

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

Thank you!

candrews avatar Oct 22 '24 20:10 candrews

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

saghul avatar Oct 22 '24 21:10 saghul

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.

ChrisLovering avatar Oct 24 '24 16:10 ChrisLovering

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

Zaczero avatar Nov 05 '24 20:11 Zaczero

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

No767 avatar Nov 08 '24 02:11 No767

Could you please upload CPython 3.13 wheels to PyPI?

monosans avatar Nov 10 '24 18:11 monosans

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

ChrisLovering avatar Nov 18 '24 19:11 ChrisLovering

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.

saghul avatar Nov 23 '24 08:11 saghul

Hey, @saghul. I've created a PR which should fix the CI release and also added more platforms to cibuildwheel. See https://github.com/saghul/pycares/pull/204.

monosans avatar Nov 25 '24 07:11 monosans