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 support for Python 3.11 and PyPy3.9 #564

Merged
merged 5 commits into from Sep 15, 2022
Merged

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Sep 14, 2022

Changes proposed in this pull request:

  • Support Python 3.11:
    • add Trove classifier
    • test on CI
    • build wheel
  • Create wheel for PyPy3.9

Python 3.11 is currently at release candidate 2, the last one before full release on 2022-10-24, and includes this call to action:

Community members
We strongly encourage maintainers of third-party Python projects to prepare their projects for 3.11 compatibilities during this phase. As always, report any issues to the Python bug tracker 1.

And:

There will be no ABI changes from this point forward in the 3.11 series and the goal is that there will be as few code changes as possible.

So it is pretty safe to release a 3.11 release now. We can label it as a dev preview, and put also out a new release when 3.11.0 is out.

But also, if you'd prefer to wait until the full 3.11.0 release, that's fine too, we've not yet had anyone asking for 3.11 wheels yet.


Also in this PR:

  • Bump GitHub Actions versions
  • GHA: Use ubuntu-latest, except pin to latest 22.04 for the benchmarking

@hugovk hugovk added the changelog: Added For new features label Sep 14, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #564 (b5ce312) into main (203f311) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #564   +/-   ##
=======================================
  Coverage   91.58%   91.58%           
=======================================
  Files           6        6           
  Lines        1901     1902    +1     
=======================================
+ Hits         1741     1742    +1     
  Misses        160      160           
Impacted Files Coverage Δ
tests/test_ujson.py 99.45% <0.00%> (+<0.01%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Collaborator

@bwoodsend bwoodsend left a comment

Choose a reason for hiding this comment

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

RC2 is close enough to the real thing for me. Bombs away...

Copy link
Collaborator

@JustAnotherArchivist JustAnotherArchivist left a comment

Choose a reason for hiding this comment

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

Normal release without a dev preview mark or similar sounds good to me.
Other changes look fine as well, thanks!

@JustAnotherArchivist
Copy link
Collaborator

I'm getting a red X for the CI on b5ce312 for whatever reason even though all checks passed. Just GitHub Actions being weird?

@bwoodsend
Copy link
Collaborator

A red X which, if you click on it, creates a popup showing that all 36 checks passed. 🙄 Is it just me or is GH's web UI getting buggier and buggier...

@hugovk
Copy link
Member Author

hugovk commented Sep 15, 2022

I think the red X is because I added the "changelog" label after creating the PR:

image

So the "Require PR label" action failed, as expected. Then I added it, and it re-ran, and passed, as expected:

image

But it makes no sense to still show the red X if it later passed... 🤷

@hugovk hugovk merged commit d84c832 into ultrajson:main Sep 15, 2022
@hugovk hugovk deleted the add-3.11 branch September 15, 2022 11:22
@hugovk
Copy link
Member Author

hugovk commented Sep 15, 2022

OK, all good to release with these release notes?

image

https://github.com/ultrajson/ultrajson/releases

@bwoodsend
Copy link
Collaborator

Yay! I did something in this one 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: Added For new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants