Skip to content

User saner python version syntax #959

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

Merged
merged 2 commits into from
Jul 11, 2025
Merged

Conversation

dandavison
Copy link
Contributor

The previous syntax was very obscure (apparently it meant 3.9 <= x < 4) and latest version of uv started to complain about it.

@dandavison dandavison requested a review from a team as a code owner July 11, 2025 02:20
@dandavison dandavison force-pushed the pyproject-requires-python branch from 6160694 to dc49129 Compare July 11, 2025 16:05
pyproject.toml Outdated
@@ -3,7 +3,7 @@ name = "temporalio"
version = "1.14.1"
description = "Temporal.io Python SDK"
authors = [{ name = "Temporal Technologies Inc", email = "[email protected]" }]
requires-python = "~=3.9"
requires-python = ">=3.9, <=3.13"
Copy link
Member

@cretz cretz Jul 11, 2025

Choose a reason for hiding this comment

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

Do we want to disallow 3.14? I would like to leave max open ended and if we are concerned about next Python, we solve that (I know we did some stuff for #863). Would even be ok w/ a single CI run for the next beta Python if easy to setup.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes you're right, there was no reason to change the semantics; just the syntax. Although I have now changed semantics to allow Python 4 as you propose.

Copy link
Member

@cretz cretz Jul 11, 2025

Choose a reason for hiding this comment

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

👍 I suspect we'll all be long gone before Python 4 ever appears, because I would assume every human that was around during the 2-to-3 transition would not stand idly by and let it happen again :-)

@dandavison dandavison merged commit 28f43f1 into main Jul 11, 2025
26 of 28 checks passed
@dandavison dandavison deleted the pyproject-requires-python branch July 11, 2025 18:58
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.

3 participants