Skip to content

Sort Simple API detail page releases#1272

Draft
jobselko wants to merge 1 commit into
pulp:mainfrom
jobselko:sort_simple_api
Draft

Sort Simple API detail page releases#1272
jobselko wants to merge 1 commit into
pulp:mainfrom
jobselko:sort_simple_api

Conversation

@jobselko

Copy link
Copy Markdown
Member

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Assisted By: Claude Opus 4.6
@jobselko

Copy link
Copy Markdown
Member Author

Behaviour before this patch: https://packages.redhat.com/trusted-libraries/python/aiobotocore/

@gerrod3 gerrod3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was this requested by someone? Not sure it is needed, from the PEP:

Note that because servers may hold “legacy” data from before the adoption of PEP 440, version strings currently cannot be required to be valid PEP 440 versions, and therefore cannot be assumed to be orderable using the PEP 440 rules. However, servers SHOULD use normalised PEP 440 versions where possible.

https://peps.python.org/pep-0700/#versions

@jobselko

jobselko commented Jul 15, 2026

Copy link
Copy Markdown
Member Author

@gerrod3 This was not requested, but when I started working on the yank feature, I noticed there were tens of unsorted files in Simple API and was not able to find the latest release because of this. I also remember that this used to be sorted some time ago (we did a naive sorting, for example: twine-1.1.0.tar.gz, twine-1.10.0.tar.gz, twine-1.2.0.tar.gz instead of twine-1.1.0.tar.gz, twine-1.2.0.tar.gz, twine-1.10.0.tar.gz). I will take a look at PEP-700 tomorrow.

Edit: https://pypi.org/simple/twine/ is sorted, so I would like our Simple API to be sorted as well.

@jobselko jobselko marked this pull request as draft July 15, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants