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

setup: remove upper bound from python_requires #9685

Merged

Conversation

vfazio
Copy link

@vfazio vfazio commented Jun 18, 2024

A cap was recently added to denote the upper bound of python support for pydantic v1.

Further research, however, has determined that python_requires was not developed with an upper bound in mind 1 and may actually cause some dependency solvers to silently fall back to older versions where there is no cap, and thus presumed support for the user's python.

For now, remove the upper bound.

Change Summary

Related issue number

fix #9663

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

skip change file check

Footnotes

  1. https://discuss.python.org/t/requires-python-upper-limits/12663

A cap was recently added to denote the upper bound of python support for
pydantic v1.

Further research, however, has determined that `python_requires` was not
developed with an upper bound in mind [0] and may actually cause some
dependency solvers to silently fall back to older versions where there
is no cap, and thus presumed support for the user's python.

For now, remove the upper bound.

[0]: https://discuss.python.org/t/requires-python-upper-limits/12663

Signed-off-by: Vincent Fazio <[email protected]>
@vfazio vfazio marked this pull request as ready for review June 18, 2024 17:09
Copy link
Member

@sydney-runkle sydney-runkle left a comment

Choose a reason for hiding this comment

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

Looks great, assuming CI is happy. Thank you!

@sydney-runkle sydney-runkle enabled auto-merge (squash) June 18, 2024 17:11
@sydney-runkle sydney-runkle merged commit 8f84b49 into pydantic:1.10.X-fixes Jun 18, 2024
59 checks passed
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