-
Notifications
You must be signed in to change notification settings - Fork 2.5k
chore: prevent normalization of semver versioning #1292
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
This PR is on hold as @dandhlee confirmed that this requires a minimum version of |
All is good now, if there is no problem with Kokoro pre-submits then it's fine, otherwise I'm working through the repositories to make necessary change and workarounds. |
@dandhlee This repo doesn't declare You won't see a failure in CI since the tests will install the latest version of |
Understood, will make those changes around! Sorry for that 😓 |
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.
LGTM
When there is a patch version added to semver versioning, setuptools.setup(version) will normalize the versioning from
-patch
to.patch
which is not correct SEMVER versioning. The added feature with setuptools.sic(version) will prevent this from happening.