Upgrade pyo3 to 0.17 and bump pyo3_asyncio version

awestlake87 / pyo3-asyncio

Other

312 stars 48 forks source link

Upgrade pyo3 to 0.17 and bump pyo3_asyncio version #76

Closed tristan closed 2 years ago

tristan commented 2 years ago

Done:

upgrade pyo3 dependency to 0.17
upgrade inventory to 0.3 (to match pyo3)
fixed tests
fixed CI to deal with messy MSRV requirements with inventory upgrade (copied and modified from pyo3's handling)
added running tests to CI (if there was a reason these weren't run previously, i can remove them as well)
removed a bunch of end of line whitespace (simply because my editor did it automatically and I was lazy to undo it, if i should remove them I can)

awestlake87 commented 2 years ago

Thanks for the PR @tristan! Just at a glance this looks good to me. I think there are some spots in the docs that might need to be touched up before it goes live (no need to worry about that), but I'll try to get everything wrapped up and released within a few days!

tristan commented 2 years ago

Ugh, I hadn't tested all the environments properly after I added the testing phase. I disabled them for platforms that don't support the current test suite (e.g. prepare_freethreaded_python is only available in non-pypy environments so the tests failed to compile at all in those environments). Also the tests for x86 windows needed the --target flag to run them as the correct architecture.

The tests run to completion on my forked repo, so :crossed_fingers: it will pass on here too this time