Merge pull request #975 from endjin/feature/974

Exclude base commit when getting intermediate commits.

Author: JakeGinnivan

src/GitVersionCore.Tests/IntegrationTests/VersionBumpingScenarios.cs

@@ -43,4 +43,24 @@ public void CanUseCommitMessagesToBumpVersion()
         }
 
     }
+
+    [Test]
+    public void CanUseCommitMessagesToBumpVersionBaseVersionTagIsAppliedToSameCommit()
+    {
+        using (var fixture = new EmptyRepositoryFixture())
+        {
+            fixture.Repository.MakeACommit();
+            fixture.MakeATaggedCommit("1.0.0");
+            fixture.Repository.MakeACommit("+semver:minor");
+            fixture.AssertFullSemver("1.1.0+1");
+
+            fixture.ApplyTag("2.0.0");
+
+            fixture.Repository.MakeACommit("Hello");
+
+            // Default bump is patch
+
+            fixture.AssertFullSemver("2.0.1+1");
+        }
+    }
 }

src/GitVersionCore/IncrementStrategyFinder.cs

@@ -104,11 +104,11 @@ private static IEnumerable<Commit> GetIntermediateCommits(IRepository repo, Comm
             var found = false;
             foreach (var commit in intermediateCommitCache)
             {
-                if (commit.Sha == baseCommit.Sha)
-                    found = true;
-
                 if (found)
                     yield return commit;
+
+                if (commit.Sha == baseCommit.Sha)
+                    found = true;
             }
         }