search

adamreeve / semver.net

Semantic versioning for .NET
MIT License
117 stars 22 forks source link

fix: 3.2.x should satisfied by 3.2.0-beta.1 #55

Closed anatawa12 closed 1 year ago

anatawa12 commented 1 year ago

Fixes #54

adamreeve commented 1 year ago

Hi, thanks for raising this issue and creating a PR with the fix. I'm a little wary this could be a breaking change for some users although it's probably reasonable to consider this as a bugfix. This matches node-semver behaviour and seems to be a more correct interpretation of how includePrerelease should work to me.

adamreeve commented 1 year ago

After thinking about this a little more and seeing that #56 also changes caret range behaviour, I think it will make sense to release this in a new major version. As although these PRs could be considered bug fixes they could still require code changes for users relying on the existing behaviour.