Merge pull request #231 from JakeGinnivan/TagComparisons

Tag comparison was treating stable as higher than pre-release causing pa...

Author: JakeGinnivan

GitVersionCore/SemanticVersionPreReleaseTag.cs

@@ -97,6 +97,15 @@ public static SemanticVersionPreReleaseTag Parse(string preReleaseTag)
 
         public int CompareTo(SemanticVersionPreReleaseTag other)
         {
+            if (!HasTag() && other.HasTag())
+            {
+                return 1;
+            }
+            if (HasTag() && !other.HasTag())
+            {
+                return -1;
+            }
+
             var nameComparison = StringComparer.InvariantCultureIgnoreCase.Compare(Name, other);
             if (nameComparison != 0)
                 return nameComparison;

Tests/SemanticVersionTests.cs

@@ -61,6 +61,12 @@ public void LegacySemVerTest()
         new SemanticVersionPreReleaseTag("AReallyReallyReallyLongBranchName", 1).ToString("lp").ShouldBe("AReallyReallyRea0001");
     }
 
+    [Test]
+    public void VersionSorting()
+    {
+        SemanticVersion.Parse("1.0.0").ShouldBeGreaterThan(SemanticVersion.Parse("1.0.0-beta"));
+    }
+
     [Test]
     public void EmptyVersion()
     {