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
fix(versioning/poetry): version strings that include a leading zero in their pre, post, or dev fields #28158
Conversation
Head branch was pushed to by a user without write access
sorry 🤦 I clearly can't read instructions and didn't realize this had already been looked at before I pushed a small change to the tests |
@rarkins sorry I pushed I hadn't realized you had already taken a look. It was a tiny change to the tests to make them look more similar in style. |
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.
see comment
🎉 This PR is included in version 37.280.4 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
Allows parsing of python version strings that include a leading zero in their pre, post, or dev fields (allowed by Python but not by Semver). An example would be 0.3.0b01 which is valid but would fail to parse under the old code.
Context
Our self hosted renovate was failing to update one package that uses this versioning scheme
Documentation (please check one with an [x])
[ ] I have updated the documentation, or
[x] No documentation update is required
How I've tested my work (please select one)
I have verified these changes via:
[] Code inspection only, or
[x] Newly added/modified unit tests, or
[ ] No unit tests but ran on a real repository, or
[ ] Both unit tests + ran on a real repository