Skip to content
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

Implement SemVer strict parsing #3131

Merged
merged 1 commit into from
May 31, 2022

Conversation

arturcic
Copy link
Member

This PR adds 2 ways of parsing the semver string into SemanticVersion object - Loose - the old way we used so far, and Strict using the regex from https://semver.org.

Updated the tests to use the strict mode by default.

@arturcic arturcic requested a review from asbjornu May 30, 2022 07:48
Copy link
Member

@asbjornu asbjornu left a comment

Choose a reason for hiding this comment

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

Looks good, only a little nit on the documentation.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@arturcic arturcic force-pushed the feature/strict-semver branch from 6abe670 to 90691a3 Compare May 31, 2022 07:14
@arturcic arturcic added this to the 6.x milestone May 31, 2022
@arturcic arturcic merged commit 0c4010c into GitTools:main May 31, 2022
@arturcic arturcic deleted the feature/strict-semver branch May 31, 2022 07:46
@mergify
Copy link
Contributor

mergify bot commented May 31, 2022

Thank you @arturcic for your contribution!

@arturcic arturcic modified the milestones: 6.x, 6.0.0-alpha.1 Dec 22, 2022
@arturcic
Copy link
Member Author

🎉 This issue has been resolved in version 6.0.0 🎉
The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants