Fix SemVerHelper

[omares]

Fix for #847

• Added active pattern for semver matching
• Fixed parsing by utilizing SemVer active pattern
• Fixed ToString conversion

[omares]

Update:

• Added PreRelease parsing according to SemVer specification
• Removed Build from PreRelease type as it is not needed
• Fixed Tests

As mentioned in chat i do not know why the when_parsing_many_pre_release_versions test fails, as it is not comprehensible to me. Imho the expectations are not correct.

[forki]

will look into this on monday. thx a lot

[omares]

@forki as mentioned in chat i would be willing to finish this PR when its clear how to proceed :) the helper still includes a small issue i will fix on tuesday.

[matthid]

Can you try to send the PR on top of my matthid:coreclr branch? This would make sure your changes are not lost in the new netcore release (and you could use them right away from the AppVeyor nuget feed (https://ci.appveyor.com/nuget/fake). If you need help, let me know.

Thanks!

[matthid]

There is also another problem with this. nuget.org is still slowly migrating to semver v2 and this change might break users of nuget packages published with FAKE... However I still think this is the correct fix. We probably apply this and look where things break...
It would probably be a good idea to provide the old behavior via an obsolete member.

[matthid]

This probably needs some re-basing. Note that I would be much more comfortable with adding this to the new Fake.Core.SemVer package instead of the legacy fake4 code (considering potential breakages).
Maybe related fsprojects/Paket#2319

[matthid]

Leaving open because of the reported bug.

[matthid]

I applied this patch to Fake.Core.SemVer, which is the new FAKE 5 module. This removes the headache of maybe introducing a breaking change. Therefore I close this due to inactivity. Thanks for the submission!