-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Requires-Python error message should sort versions numerically, not alphabetically #13394
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
Conversation
Remove changes to the vendored code, it doesn't look relevant, we don't change that code directly in pip it's changed in the upstream package, and the changes you are making are violating the spec (PEP 440). |
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have used a lambda rather than an inline function, but that's mostly a stylistic choice, no objections apart from updating the news item.
Implements #13367
Requires-Python is str, so I converted it to a list of Versions and sorted it.