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

Remove upper constraint on Python < 4 #239

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

edgarrmondragon
Copy link

@edgarrmondragon edgarrmondragon commented Oct 27, 2024

It causes problems for downstream dependents, even indirectly, specially
for Poetry. Most recently:

Because my-package depends on rsa (4.7) which requires Python >=3.5, <4, version solving failed.

  • Check your dependencies Python requirement: The Python requirement can be specified via the `python` or `markers` properties

    For rsa, a possible solution would be to set the `python` property to ">=3.8,<4"

    https://python-poetry.org/docs/dependency-specification/#python-restricted-dependencies,
    https://python-poetry.org/docs/dependency-specification/#using-environment-markers

From the comments in the Discourse by Core developers and SC members, there probably won't be a Python 4 so I think it's fine to remove this constraint.

It causes problems for downstream dependents, even indirectly, specially
for Poetry. Most recently:

```
Because my-package depends on rsa (4.7) which requires Python >=3.5, <4, version solving failed.

  • Check your dependencies Python requirement: The Python requirement can be specified via the `python` or `markers` properties

    For rsa, a possible solution would be to set the `python` property to ">=3.8,<4"

    https://python-poetry.org/docs/dependency-specification/#python-restricted-dependencies,
    https://python-poetry.org/docs/dependency-specification/#using-environment-markers
```

From the comments in the [Discourse](https://discuss.python.org) by Core
developers and SC members, there probably won't a Python 4 so I think
it's fine to remove this constraint.
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.

1 participant