From 28fbd89102c709f6a653bb748d698b56c4434650 Mon Sep 17 00:00:00 2001 From: Frank Genois Date: Thu, 31 Mar 2022 17:20:55 -0400 Subject: [PATCH 1/8] The Calculation of BaseVersions should be based on tags first, then merge commits --- .../IntegrationTests/DevelopScenarios.cs | 24 ++++---- .../IntegrationTests/DocumentationSamples.cs | 4 +- .../IntegrationTests/GitflowScenarios.cs | 12 ++-- .../ReleaseBranchScenarios.cs | 32 +++++++++-- .../BaseVersionCalculator.cs | 57 +++++++++++++++---- 5 files changed, 93 insertions(+), 36 deletions(-) diff --git a/src/GitVersion.Core.Tests/IntegrationTests/DevelopScenarios.cs b/src/GitVersion.Core.Tests/IntegrationTests/DevelopScenarios.cs index bd9f36b2a4..b63e2e43a7 100644 --- a/src/GitVersion.Core.Tests/IntegrationTests/DevelopScenarios.cs +++ b/src/GitVersion.Core.Tests/IntegrationTests/DevelopScenarios.cs @@ -228,11 +228,11 @@ public void CommitsSinceVersionSourceShouldNotGoDownUponGitFlowReleaseFinish() fixture.Checkout("develop"); fixture.MergeNoFF("release/1.2.0"); fixture.MakeACommit("commit in develop - 6"); - fixture.AssertFullSemver("1.3.0-alpha.9"); + fixture.AssertFullSemver("1.3.0-alpha.6"); fixture.SequenceDiagram.Destroy("release/1.2.0"); fixture.Repository.Branches.Remove("release/1.2.0"); - const string expectedFullSemVer = "1.3.0-alpha.9"; + const string expectedFullSemVer = "1.3.0-alpha.6"; fixture.AssertFullSemver(expectedFullSemVer, config); } @@ -260,11 +260,11 @@ public void CommitsSinceVersionSourceShouldNotGoDownUponMergingFeatureOnlyToDeve fixture.MakeACommit("commit in develop - 2"); fixture.AssertFullSemver("1.3.0-alpha.1"); fixture.MergeNoFF("release/1.2.0"); - fixture.AssertFullSemver("1.3.0-alpha.5"); + fixture.AssertFullSemver("1.3.0-alpha.2"); fixture.SequenceDiagram.Destroy("release/1.2.0"); fixture.Repository.Branches.Remove("release/1.2.0"); - const string expectedFullSemVer = "1.3.0-alpha.5"; + const string expectedFullSemVer = "1.3.0-alpha.2"; fixture.AssertFullSemver(expectedFullSemVer, config); } @@ -317,16 +317,16 @@ public void WhenPreventIncrementOfMergedBranchVersionIsSetToFalseForDevelopCommi fixture.MergeNoFF(ReleaseBranch); // Version numbers will still be correct when the release branch is around. - fixture.AssertFullSemver("1.2.0-alpha.6"); - fixture.AssertFullSemver("1.2.0-alpha.6", config); + fixture.AssertFullSemver("1.2.0-alpha.3"); + fixture.AssertFullSemver("1.2.0-alpha.3", config); var versionSourceBeforeReleaseBranchIsRemoved = fixture.GetVersion(config).Sha; fixture.Repository.Branches.Remove(ReleaseBranch); var versionSourceAfterReleaseBranchIsRemoved = fixture.GetVersion(config).Sha; Assert.AreEqual(versionSourceBeforeReleaseBranchIsRemoved, versionSourceAfterReleaseBranchIsRemoved); - fixture.AssertFullSemver("1.2.0-alpha.6"); - fixture.AssertFullSemver("1.2.0-alpha.6", config); + fixture.AssertFullSemver("1.2.0-alpha.3"); + fixture.AssertFullSemver("1.2.0-alpha.3", config); config.Branches = new Dictionary { @@ -374,7 +374,7 @@ public void WhenPreventIncrementOfMergedBranchVersionIsSetToFalseForDevelopCommi fixture.Repository.MakeCommits(2); fixture.MergeNoFF(ReleaseBranch); fixture.Repository.Branches.Remove(ReleaseBranch); - fixture.AssertFullSemver("1.2.0-alpha.6", config); + fixture.AssertFullSemver("1.2.0-alpha.3", config); // Create hotfix for defects found in release/1.1.0 const string HotfixBranch = "hotfix/1.1.1"; @@ -391,11 +391,11 @@ public void WhenPreventIncrementOfMergedBranchVersionIsSetToFalseForDevelopCommi fixture.Checkout("develop"); // Simulate some work done on develop while the hotfix branch was open. fixture.Repository.MakeCommits(3); - fixture.AssertFullSemver("1.2.0-alpha.9", config); + fixture.AssertFullSemver("1.2.0-alpha.6", config); fixture.Repository.MergeNoFF(HotfixBranch); - fixture.AssertFullSemver("1.2.0-alpha.19", config); + fixture.AssertFullSemver("1.2.0-alpha.10", config); fixture.Repository.Branches.Remove(HotfixBranch); - fixture.AssertFullSemver("1.2.0-alpha.19", config); + fixture.AssertFullSemver("1.2.0-alpha.10", config); } } diff --git a/src/GitVersion.Core.Tests/IntegrationTests/DocumentationSamples.cs b/src/GitVersion.Core.Tests/IntegrationTests/DocumentationSamples.cs index 5b1d12634a..2de00f5b6d 100644 --- a/src/GitVersion.Core.Tests/IntegrationTests/DocumentationSamples.cs +++ b/src/GitVersion.Core.Tests/IntegrationTests/DocumentationSamples.cs @@ -155,7 +155,7 @@ public void GitFlowMinorRelease() // Not 0 for commit count as we can't know the increment rules of the merged branch fixture.Checkout("develop"); - fixture.AssertFullSemver("1.4.0-alpha.4"); + fixture.AssertFullSemver("1.4.0-alpha.2"); Console.WriteLine(fixture.SequenceDiagram.GetDiagram()); } @@ -211,7 +211,7 @@ public void GitFlowMajorRelease() // Not 0 for commit count as we can't know the increment rules of the merged branch fixture.Checkout("develop"); - fixture.AssertFullSemver("2.1.0-alpha.4"); + fixture.AssertFullSemver("2.1.0-alpha.2"); Console.WriteLine(fixture.SequenceDiagram.GetDiagram()); } diff --git a/src/GitVersion.Core.Tests/IntegrationTests/GitflowScenarios.cs b/src/GitVersion.Core.Tests/IntegrationTests/GitflowScenarios.cs index 630400c766..4c887a994b 100644 --- a/src/GitVersion.Core.Tests/IntegrationTests/GitflowScenarios.cs +++ b/src/GitVersion.Core.Tests/IntegrationTests/GitflowScenarios.cs @@ -41,16 +41,16 @@ public void GitflowComplexExample() fixture.Checkout(developBranch); fixture.MergeNoFF(release1Branch); fixture.Repository.Branches.Remove(fixture.Repository.Branches[release1Branch]); - fixture.AssertFullSemver("1.2.0-alpha.2"); + fixture.AssertFullSemver("1.2.0-alpha.1"); // Feature 2 fixture.BranchTo(feature2Branch); fixture.MakeACommit("added feature 2"); - fixture.AssertFullSemver("1.2.0-f2.1+3"); + fixture.AssertFullSemver("1.2.0-f2.1+2"); fixture.Checkout(developBranch); fixture.MergeNoFF(feature2Branch); fixture.Repository.Branches.Remove(fixture.Repository.Branches[feature2Branch]); - fixture.AssertFullSemver("1.2.0-alpha.4"); + fixture.AssertFullSemver("1.2.0-alpha.3"); // Release 1.2.0 fixture.BranchTo(release2Branch); @@ -64,13 +64,13 @@ public void GitflowComplexExample() fixture.Checkout(developBranch); fixture.MergeNoFF(release2Branch); fixture.Repository.Branches.Remove(fixture.Repository.Branches[release2Branch]); - fixture.AssertFullSemver("1.3.0-alpha.2"); + fixture.AssertFullSemver("1.3.0-alpha.1"); // Hotfix fixture.Checkout(MainBranch); fixture.BranchTo(hotfixBranch); fixture.MakeACommit("added hotfix"); - fixture.AssertFullSemver("1.2.1-beta.1+7"); + fixture.AssertFullSemver("1.2.1-beta.1+1"); fixture.Checkout(MainBranch); fixture.MergeNoFF(hotfixBranch); fixture.AssertFullSemver("1.2.1+2"); @@ -79,7 +79,7 @@ public void GitflowComplexExample() fixture.Checkout(developBranch); fixture.MergeNoFF(hotfixBranch); fixture.Repository.Branches.Remove(fixture.Repository.Branches[hotfixBranch]); - fixture.AssertFullSemver("1.3.0-alpha.9"); + fixture.AssertFullSemver("1.3.0-alpha.3"); } } } diff --git a/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs b/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs index 0fd75f14a8..a54c2ec216 100644 --- a/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs +++ b/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs @@ -48,12 +48,12 @@ public void NoMergeBacksToDevelopInCaseThereAreChangesInReleaseBranch() // Merge to develop fixture.Checkout("develop"); fixture.Repository.MergeNoFF("release/1.0.0"); - fixture.AssertFullSemver("1.1.0-alpha.2"); + fixture.AssertFullSemver("1.1.0-alpha.1"); fixture.Repository.MakeACommit(); fixture.Repository.Branches.Remove("release/1.0.0"); - fixture.AssertFullSemver("1.1.0-alpha.3"); + fixture.AssertFullSemver("1.1.0-alpha.2"); } [Test] @@ -184,7 +184,7 @@ public void WhenReleaseBranchOffMainIsMergedIntoMainVersionIsTakenWithIt() } [Test] - public void MainVersioningContinuousCorrectlyAfterMergingReleaseBranch() + public void MainVersioningContinuesCorrectlyAfterMergingReleaseBranch() { using var fixture = new EmptyRepositoryFixture(); fixture.Repository.MakeATaggedCommit("1.0.3"); @@ -201,6 +201,30 @@ public void MainVersioningContinuousCorrectlyAfterMergingReleaseBranch() fixture.AssertFullSemver("2.0.1+1"); } + [Test] + public void MainVersioningContinuesCountingAfterMergingTheSameReleaseBranchMultipleTimes() + { + using var fixture = new EmptyRepositoryFixture(); + fixture.Repository.MakeATaggedCommit("1.0.3"); + fixture.Repository.MakeCommits(1); + fixture.Repository.CreateBranch("release/2.0.0"); + fixture.Checkout("release/2.0.0"); + fixture.Repository.MakeCommits(4); + fixture.Checkout(MainBranch); + fixture.Repository.MergeNoFF("release/2.0.0", Generate.SignatureNow()); + + fixture.AssertFullSemver("2.0.0+0"); + + fixture.Checkout("release/2.0.0"); + fixture.Repository.MakeCommits(3); + fixture.Checkout(MainBranch); + fixture.Repository.MergeNoFF("release/2.0.0", Generate.SignatureNow()); + + fixture.AssertFullSemver("2.0.0+4"); + fixture.Repository.MakeCommits(1); + fixture.AssertFullSemver("2.0.0+5"); + } + [Test] public void WhenReleaseBranchIsMergedIntoDevelopHighestVersionIsTakenWithIt() { @@ -434,7 +458,7 @@ public void CommitOnDevelopAfterReleaseBranchMergeToDevelopShouldNotResetCount() } [Test] - public void CommitBeetweenMergeReleaseToDevelopShouldNotResetCount() + public void CommitBetweenMergeReleaseToDevelopShouldNotResetCount() { var config = new Config { diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index 62542bad17..8a4838ff25 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -53,24 +53,53 @@ public BaseVersion GetBaseVersion() if (matchingVersionsOnceIncremented.Any()) { - static Versions CompareVersions(Versions versions1, Versions version2) + if (matchingVersionsOnceIncremented.Count > 1) { - if (versions1.Version.BaseVersionSource == null) + static Versions CompareVersions(Versions versions1, Versions version2) { - return version2; + if (versions1.Version.BaseVersionSource == null) + { + return version2; + } + if (version2.Version.BaseVersionSource == null) + { + return versions1; + } + + return versions1.Version.BaseVersionSource.When < version2.Version.BaseVersionSource.When ? versions1 : version2; } - if (version2.Version.BaseVersionSource == null) + + log.Info($"Found multiple base versions which will produce the same SemVer ({maxVersion.IncrementedVersion})"); + log.Info($"Here are the different source candidate for commit counting : "); + foreach (var baseVersion in matchingVersionsOnceIncremented.Select(b => b.Version)) { - return versions1; + if (baseVersion != null) + { + log.Info($" - {BaseVersionToString(baseVersion)}"); + } } - return versions1.Version.BaseVersionSource.When < version2.Version.BaseVersionSource.When ? versions1 : version2; - } + var tagVersions = matchingVersionsOnceIncremented.FindAll(b => b.Version.Source.Contains("Git tag")); + + if (tagVersions.Count > 0) + { + log.Info("As there are Git tags, the other sources will be discarded"); + matchingVersionsOnceIncremented = tagVersions; + } - var oldest = matchingVersionsOnceIncremented.Aggregate(CompareVersions); - baseVersionWithOldestSource = oldest.Version; - maxVersion = oldest; - this.log.Info($"Found multiple base versions which will produce the same SemVer ({maxVersion.IncrementedVersion}), taking oldest source for commit counting ({baseVersionWithOldestSource.Source})"); + maxVersion = matchingVersionsOnceIncremented.Aggregate(CompareVersions); + baseVersionWithOldestSource = maxVersion.Version; + log.Info($"Taking oldest source for commit counting : {BaseVersionToString(baseVersionWithOldestSource)}"); + } + else + { + maxVersion = matchingVersionsOnceIncremented.First(); + baseVersionWithOldestSource = maxVersion.Version; + log.Info( + $"Found a base versions which will produce the following SemVer ({maxVersion.IncrementedVersion}), " + + $"with the following source for commit counting : {BaseVersionToString(baseVersionWithOldestSource)}" + ); + } } else { @@ -92,11 +121,15 @@ static Versions CompareVersions(Versions versions1, Versions version2) baseVersionWithOldestSource.BaseVersionSource, maxVersion.Version.BranchNameOverride); - this.log.Info($"Base version used: {calculatedBase}"); + log.Info($"Base version used: {calculatedBase}"); return calculatedBase; } } + + private static string BaseVersionToString(BaseVersion baseVersion) => + $"{baseVersion!.Source} ({baseVersion!.BaseVersionSource!.Sha})"; + private IEnumerable GetBaseVersions(IVersionStrategy strategy) { foreach (var version in strategy.GetVersions()) From 9c6fbddd844bc8e7275f5899bd927d5a858a5ea7 Mon Sep 17 00:00:00 2001 From: Frank Genois Date: Mon, 4 Apr 2022 09:54:59 -0400 Subject: [PATCH 2/8] Added test that proves #2394 has been fixed Fixes #2394 --- .../ReleaseBranchScenarios.cs | 27 +++++++++++++++++++ 1 file changed, 27 insertions(+) diff --git a/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs b/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs index a54c2ec216..f55ca21fd3 100644 --- a/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs +++ b/src/GitVersion.Core.Tests/IntegrationTests/ReleaseBranchScenarios.cs @@ -6,6 +6,7 @@ using GitVersion.VersionCalculation; using LibGit2Sharp; using NUnit.Framework; +using Shouldly; namespace GitVersion.Core.Tests.IntegrationTests; @@ -225,6 +226,32 @@ public void MainVersioningContinuesCountingAfterMergingTheSameReleaseBranchMulti fixture.AssertFullSemver("2.0.0+5"); } + [Test] + public void ShouldUseTagsAsVersionSourceWhenPossible() + { + using var fixture = new EmptyRepositoryFixture(); + fixture.Repository.MakeATaggedCommit("0.1.0"); + fixture.Checkout(MainBranch); + fixture.Repository.CreateBranch("develop"); + fixture.Checkout("develop"); + fixture.MakeACommit("Feature commit 1"); + fixture.BranchTo("release/0.2.0"); + fixture.MakeACommit("Release commit 1"); + fixture.Checkout(MainBranch); + fixture.MergeNoFF("release/0.2.0"); + fixture.ApplyTag("0.2.0"); + + var tag = fixture.Repository.Head.Tip; + + fixture.Checkout("develop"); + fixture.MergeNoFF(MainBranch); + + fixture.AssertFullSemver("0.3.0-alpha.1"); + + var version = fixture.GetVersion(); + version.VersionSourceSha.ShouldBeEquivalentTo(tag.Sha); + } + [Test] public void WhenReleaseBranchIsMergedIntoDevelopHighestVersionIsTakenWithIt() { From 34508cb27b824df398ceaec064570669f90b6120 Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:02:58 -0400 Subject: [PATCH 3/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index 8a4838ff25..7f22d04f8a 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -121,7 +121,7 @@ static Versions CompareVersions(Versions versions1, Versions version2) baseVersionWithOldestSource.BaseVersionSource, maxVersion.Version.BranchNameOverride); - log.Info($"Base version used: {calculatedBase}"); + this.log.Info($"Base version used: {calculatedBase}"); return calculatedBase; } From 2dd89a8669a4ae9ed46a37a7bf028b5487d95d8f Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:03:09 -0400 Subject: [PATCH 4/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- .../VersionCalculation/BaseVersionCalculator.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index 7f22d04f8a..6ead9f5783 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -69,8 +69,8 @@ static Versions CompareVersions(Versions versions1, Versions version2) return versions1.Version.BaseVersionSource.When < version2.Version.BaseVersionSource.When ? versions1 : version2; } - log.Info($"Found multiple base versions which will produce the same SemVer ({maxVersion.IncrementedVersion})"); - log.Info($"Here are the different source candidate for commit counting : "); + this.log.Info($"Found multiple base versions which will produce the same SemVer ({maxVersion.IncrementedVersion})"); + this.log.Info($"Here are the different source candidate for commit counting : "); foreach (var baseVersion in matchingVersionsOnceIncremented.Select(b => b.Version)) { if (baseVersion != null) From 17b69b0c05f2b8a6e3193bbeca0e596695481e84 Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:03:15 -0400 Subject: [PATCH 5/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index 6ead9f5783..c1899078a0 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -95,7 +95,7 @@ static Versions CompareVersions(Versions versions1, Versions version2) { maxVersion = matchingVersionsOnceIncremented.First(); baseVersionWithOldestSource = maxVersion.Version; - log.Info( + this.log.Info( $"Found a base versions which will produce the following SemVer ({maxVersion.IncrementedVersion}), " + $"with the following source for commit counting : {BaseVersionToString(baseVersionWithOldestSource)}" ); From edf70c9f9ff69151abcea36fa2c8616a7eadded3 Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:03:21 -0400 Subject: [PATCH 6/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index c1899078a0..d527f6bb20 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -83,7 +83,7 @@ static Versions CompareVersions(Versions versions1, Versions version2) if (tagVersions.Count > 0) { - log.Info("As there are Git tags, the other sources will be discarded"); + this.log.Info("As there are Git tags, the other sources will be discarded"); matchingVersionsOnceIncremented = tagVersions; } From 7fbcbd9ad794cbe107dd9b3ad3345166c7c367d1 Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:03:29 -0400 Subject: [PATCH 7/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index d527f6bb20..c9bde5f1c6 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -75,7 +75,7 @@ static Versions CompareVersions(Versions versions1, Versions version2) { if (baseVersion != null) { - log.Info($" - {BaseVersionToString(baseVersion)}"); + this.log.Info($" - {BaseVersionToString(baseVersion)}"); } } From 7fde0805f84834e05933e36d36ac9dec2b9f3e3a Mon Sep 17 00:00:00 2001 From: ni-fgenois <60878197+ni-fgenois@users.noreply.github.com> Date: Thu, 7 Apr 2022 13:03:35 -0400 Subject: [PATCH 8/8] Update src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Asbjørn Ulsberg --- src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs index c9bde5f1c6..968f43672b 100644 --- a/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs +++ b/src/GitVersion.Core/VersionCalculation/BaseVersionCalculator.cs @@ -89,7 +89,7 @@ static Versions CompareVersions(Versions versions1, Versions version2) maxVersion = matchingVersionsOnceIncremented.Aggregate(CompareVersions); baseVersionWithOldestSource = maxVersion.Version; - log.Info($"Taking oldest source for commit counting : {BaseVersionToString(baseVersionWithOldestSource)}"); + this.log.Info($"Taking oldest source for commit counting : {BaseVersionToString(baseVersionWithOldestSource)}"); } else {