AcademySoftwareFoundation / AcademySoftwareFoundation/rez
[Feature] Support for correct pre-release version sorting
- Dominant language
- Python
- Stars
- 1.1k
- Forks
- 374
- Avg merge
- 9d 12h
- Merged PRs (30d)
- 5
Description
Both PEP 440 versions and SemVar 2.0 versions may contain pre-release (and other) version information besides just maj/min/patch.
- https://www.python.org/dev/peps/pep-0440/
- https://semver.org/
Both PEP 440 and SemVar 2.0 utilize different sorting behavior for pre-releases than the default alphanumeric sort used by rez. For example, this is the expected sort order under PEP 440:
- `1.0.0-a1`
- `1.0.0b1`
- `1.0.0.rc1`
- `1.0.0`
The default rez sort will sort them like this, with the base version *before* the pre-releases, and also doesn't support the pre-release separator being optional:
- `1.0.0`
- `1.0.0-a1`
- `1.0.0.rc1`
- `1.0.0b1`
To maintain backwards compatibility, a config option could be added to enable support for this sorting behavior -- maybe `use_pep440_versioning`? With the expectation that if the package version isn't a valid PEP 440 version, it simply falls back to the normal alphanumeric sorting behavior.
This also raises another question of whether the `version` module should be vendored or not and instead be moved into rez proper, since it's already a pretty significant fork from the original and if we want it to be config aware, it should probably be moved into rez.
It may make sense to separate this ticket (and the config option) into two different options -- one for PEP 440 and one for SemVar 2.0, since they are different enough that someone may want one behavior and not the other. For this ticket, I'm mainly concerned with PEP 440 style versioning. I included SemVar 2.0 just to show it's not an issue unique to python packaging.
Contributor guide
Research direction
Start by reviewing the existing vendored version module and Rez's current version-sorting and configuration paths. Compare the current behavior with the PEP 440 examples, then define the configuration scope and verify that valid PEP 440 versions sort correctly while invalid versions retain the existing alphanumeric fallback.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100