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

feat: add python 3.12 #2063

Closed
wants to merge 2 commits into from
Closed

feat: add python 3.12 #2063

wants to merge 2 commits into from

Conversation

JacobCoffee
Copy link
Member

@JacobCoffee JacobCoffee commented Jul 25, 2023

Pull Request Checklist

  • New code has 100% test coverage
  • (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
  • (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

  • Add Python 3.12 as a target - 3.12 Release Schedule is slated to arrive within the next 3 months, with RC 1 coming at the first of next week

Close Issue(s)

@JacobCoffee JacobCoffee self-assigned this Jul 26, 2023
@JacobCoffee JacobCoffee added Upstream This is due to or tracking an upstream issue Enhancement This is a new feature or request CI This is CI related labels Jul 26, 2023
@provinzkraut
Copy link
Member

@JacobCoffee what's the progress here? How many libs are still blocking this?

@JacobCoffee
Copy link
Member Author

Still the same - grpc.io and greenlet

@Goldziher Goldziher closed this Sep 9, 2023
@JacobCoffee JacobCoffee reopened this Sep 17, 2023
@JacobCoffee JacobCoffee marked this pull request as ready for review October 3, 2023 03:12
@JacobCoffee JacobCoffee requested review from a team as code owners October 3, 2023 03:12
Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Just a friendly reminder not to use 3.12 syntax anywhere just yet :)
Most tools with just fail to parse it, like mypy does :(

Congrats on 3.12 release 🎉

python-version: "3.11"

python-version: "3.12"
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

You can remove this line now :)
3.12.0 is final!

pydantic-version: ["1", "2"]
uses: ./.github/workflows/test.yaml
with:
coverage: ${{ matrix.python-version == '3.11' && matrix.pydantic-version == '2' }}
coverage: ${{ matrix.python-version == '3.12' && matrix.pydantic-version == '2' }}
Copy link
Member

Choose a reason for hiding this comment

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

I recommend not to use coverage with 3.12 just yet, there are some possible (and known) bugs, which can result in invalid coverage percentage or line numbers. 3.11 is fine.

python-version: "3.11"

python-version: "3.12"
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

^

@@ -14,7 +14,8 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-python@v4
with:
python-version: "3.11"
python-version: "3.12"
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

^

python-version: "3.11"

python-version: "3.12"
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

^

python-version: "3.11"

python-version: "3.12"
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

^

@@ -32,6 +32,7 @@ jobs:
uses: actions/setup-python@v4
with:
python-version: ${{ inputs.python-version }}
allow-prereleases: true
Copy link
Member

Choose a reason for hiding this comment

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

^

@JacobCoffee
Copy link
Member Author

Thanks @sobolevn! I was trying out the gh cli tool and managed to clone the pr and push it into #2396 - i applied your finding there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI This is CI related Enhancement This is a new feature or request Upstream This is due to or tracking an upstream issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement: Python 3.12
4 participants