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

Declare support for Python 3.13 #2071

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

edgarrmondragon
Copy link
Contributor

@edgarrmondragon edgarrmondragon commented Jul 9, 2024

What does this change

The first release candidate of Python 3.13 is around the corner (2024-07-30), this adds it to the test matrix and declares support in the project metadata.

What was wrong

Python 3.13 seems to be supported without any code changes, but this makes sure it's tested.

How this fixes it

  • Adds PyPI trove classifier
  • Adds 3.13 to the list of versions for actions/setup-python

@edgarrmondragon
Copy link
Contributor Author

Tests are 🟢 over on my fork: https://github.com/edgarrmondragon/faker/actions/runs/9849708765

@@ -154,6 +160,7 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python }}
allow-prereleases: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not using "3.13-dev" instead to better indicate that Python 3.13 is not yet available as a stable release?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Feels easier to let it stay as 3.13 after the final release rather than coming back and changing it from 3.13-dev to 3.13. I don't have a strong preference, though.

tox.ini Outdated Show resolved Hide resolved
@edgarrmondragon
Copy link
Contributor Author

Rebased and confirmed with tox -e py313 that tests pass.

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.

2 participants