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

SemanticVersion missing equals/not equals operator, prerelease sorted wrong #3711

Closed
devlead opened this issue Nov 23, 2021 · 1 comment · Fixed by #3712
Closed

SemanticVersion missing equals/not equals operator, prerelease sorted wrong #3711

devlead opened this issue Nov 23, 2021 · 1 comment · Fixed by #3712
Assignees
Labels
Milestone

Comments

@devlead
Copy link
Member

devlead commented Nov 23, 2021

Current behaviour

new SemanticVersion(1,0,0) == new SemanticVersion(1,0,0) == false
new SemanticVersion(1,0,0) > new SemanticVersion(1,0,0, "rc0001") == false

Expected behaviour

new SemanticVersion(1,0,0) == new SemanticVersion(1,0,0) == true
new SemanticVersion(1,0,0) > new SemanticVersion(1,0,0, "rc0001") == true

I'll send a pr shortly to address this.

@devlead devlead added the Bug label Nov 23, 2021
@devlead devlead added this to the v2.0.0 milestone Nov 23, 2021
@devlead devlead self-assigned this Nov 23, 2021
devlead added a commit to devlead/cake that referenced this issue Nov 23, 2021
* Adds SemanticVersion equals operator
* Adds SemanticVersion not equals operator
* Fixes PreRelease sorted as newer than non PreRelease
* Adds SemanticVersion tests
* Fixes cake-build#3711
@gep13 gep13 changed the title SemanticVersion missing equals/not equals operator, prerelease sortet wrong SemanticVersion missing equals/not equals operator, prerelease sorted wrong Nov 23, 2021
gep13 added a commit that referenced this issue Nov 23, 2021
GH3711: Fix SemanticVersion missing operators & comparer
@cake-build-bot
Copy link

🎉 This issue has been resolved in version v2.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 a pull request may close this issue.

2 participants