refactors tag prefix configuration

Renames `TagPrefix` to `TagPrefixPattern` for clarity.

Author: arturcic

src/GitVersion.Core.Tests/MergeMessageTests.cs

@@ -35,10 +35,10 @@ public void EmptyTagPrefix(string? prefix)
     {
         // Arrange
         const string message = "Updated some code.";
-        var configuration = GitHubFlowConfigurationBuilder.New.WithTagPrefixPattern(prefix).Build();
+        var configurationBuilder = GitHubFlowConfigurationBuilder.New.WithTagPrefixPattern(prefix);
 
         // Act
-        var sut = new MergeMessage(message, configuration);
+        var sut = new MergeMessage(message, configurationBuilder.Build());
 
         // Assert
         sut.TargetBranch.ShouldBeNull();

src/GitVersion.Core/Configuration/EffectiveConfiguration.cs

@@ -52,7 +52,7 @@ public EffectiveConfiguration(
         AssemblyVersioningFormat = configuration.AssemblyVersioningFormat;
         AssemblyFileVersioningFormat = configuration.AssemblyFileVersioningFormat;
         DeploymentMode = branchConfiguration.DeploymentMode.Value;
-        TagPrefix = configuration.TagPrefixPattern;
+        TagPrefixPattern = configuration.TagPrefixPattern;
         VersionInBranchPattern = configuration.VersionInBranchPattern;
         Label = branchConfiguration.Label;
         NextVersion = configuration.NextVersion;
@@ -90,7 +90,7 @@ public EffectiveConfiguration(
     public string? AssemblyVersioningFormat { get; }
     public string? AssemblyFileVersioningFormat { get; }
 
-    public string? TagPrefix { get; }
+    public string? TagPrefixPattern { get; }
 
     public string? VersionInBranchPattern { get; }
 

src/GitVersion.Core/Extensions/ReferenceNameExtensions.cs

@@ -6,16 +6,16 @@ namespace GitVersion.Configuration;
 
 public static class ReferenceNameExtensions
 {
-    public static bool TryGetSemanticVersion(this ReferenceName source, out (SemanticVersion Value, string? Name) result, EffectiveConfiguration configuration)
-        => source.TryGetSemanticVersion(out result, configuration.VersionInBranchPattern, configuration.TagPrefix, configuration.SemanticVersionFormat);
+    public static bool TryGetSemanticVersion(this ReferenceName source, EffectiveConfiguration configuration, out (SemanticVersion Value, string? Name) result)
+        => source.TryGetSemanticVersion(configuration.VersionInBranchPattern, configuration.TagPrefixPattern, configuration.SemanticVersionFormat, out result);
 
-    public static bool TryGetSemanticVersion(this ReferenceName source, out (SemanticVersion Value, string? Name) result, IGitVersionConfiguration configuration)
-        => source.TryGetSemanticVersion(out result, configuration.VersionInBranchPattern, configuration.TagPrefixPattern, configuration.SemanticVersionFormat);
+    public static bool TryGetSemanticVersion(this ReferenceName source, IGitVersionConfiguration configuration, out (SemanticVersion Value, string? Name) result)
+        => source.TryGetSemanticVersion(configuration.VersionInBranchPattern, configuration.TagPrefixPattern, configuration.SemanticVersionFormat, out result);
 
-    private static bool TryGetSemanticVersion(this ReferenceName referenceName, out (SemanticVersion Value, string? Name) result,
-                                              string? versionPatternPattern,
-                                              string? tagPrefix,
-                                              SemanticVersionFormat format)
+    private static bool TryGetSemanticVersion(this ReferenceName referenceName,
+                                             string? versionPatternPattern,
+                                             string? tagPrefix,
+                                             SemanticVersionFormat format, out (SemanticVersion Value, string? Name) result)
     {
         var versionPatternRegex = RegexPatterns.Cache.GetOrAdd(GetVersionInBranchPattern(versionPatternPattern));
         result = default;

src/GitVersion.Core/MergeMessage.cs

@@ -53,7 +53,7 @@ public MergeMessage(string mergeMessage, IGitVersionConfiguration configuration)
                 PullRequestNumber = pullNumber;
             }
 
-            Version = MergedBranch?.TryGetSemanticVersion(out var result, configuration) == true ? result.Value : null;
+            Version = MergedBranch?.TryGetSemanticVersion(configuration, out var result) == true ? result.Value : null;
 
             break;
         }

src/GitVersion.Core/PublicAPI.Unshipped.txt

@@ -1,4 +1,5 @@
 #nullable enable
+GitVersion.Configuration.EffectiveConfiguration.TagPrefixPattern.get -> string?
 GitVersion.Extensions.AssemblyVersionsGeneratorExtensions.extension(GitVersion.SemanticVersion!)
 GitVersion.Extensions.AssemblyVersionsGeneratorExtensions.extension(GitVersion.SemanticVersion!).GetAssemblyFileVersion(GitVersion.Configuration.AssemblyFileVersioningScheme scheme) -> string?
 GitVersion.Extensions.AssemblyVersionsGeneratorExtensions.extension(GitVersion.SemanticVersion!).GetAssemblyVersion(GitVersion.Configuration.AssemblyVersioningScheme scheme) -> string?
@@ -42,6 +43,9 @@ GitVersion.Logging.LogExtensions.extension(GitVersion.Logging.ILog!).Warning(Git
 GitVersion.Logging.LogExtensions.extension(GitVersion.Logging.ILog!).Warning(GitVersion.Logging.Verbosity verbosity, GitVersion.Logging.LogAction! logAction) -> void
 GitVersion.Logging.LogExtensions.extension(GitVersion.Logging.ILog!).Warning(GitVersion.Logging.Verbosity verbosity, string! format, params object?[]! args) -> void
 GitVersion.Logging.LogExtensions.extension(GitVersion.Logging.ILog!).Warning(string! format, params object?[]! args) -> void
+static GitVersion.Configuration.ReferenceNameExtensions.TryGetSemanticVersion(this GitVersion.Git.ReferenceName! referenceName, string? versionPatternPattern, string? tagPrefix, GitVersion.SemanticVersionFormat format, out (GitVersion.SemanticVersion! Value, string? Name) result) -> bool
+static GitVersion.Configuration.ReferenceNameExtensions.TryGetSemanticVersion(this GitVersion.Git.ReferenceName! source, GitVersion.Configuration.EffectiveConfiguration! configuration, out (GitVersion.SemanticVersion! Value, string? Name) result) -> bool
+static GitVersion.Configuration.ReferenceNameExtensions.TryGetSemanticVersion(this GitVersion.Git.ReferenceName! source, GitVersion.Configuration.IGitVersionConfiguration! configuration, out (GitVersion.SemanticVersion! Value, string? Name) result) -> bool
 static GitVersion.Extensions.FileSystemExtensions.FindGitDir(this System.IO.Abstractions.IFileSystem! fileSystem, string? path) -> (string! GitDirectory, string! WorkingTreeDirectory)?
 static GitVersion.Logging.LogExtensions.Debug(this GitVersion.Logging.ILog! log, GitVersion.Logging.Verbosity verbosity, string! format, params object?[]! args) -> void
 static GitVersion.Logging.LogExtensions.Debug(this GitVersion.Logging.ILog! log, string! format, params object?[]! args) -> void

src/GitVersion.Core/VersionCalculation/IncrementStrategyFinder.cs

@@ -73,7 +73,7 @@ public VersionField DetermineIncrementedField(
         }
 
         IEnumerable<ICommit> commits = GetCommitHistory(
-            tagPrefix: configuration.TagPrefix,
+            tagPrefix: configuration.TagPrefixPattern,
             semanticVersionFormat: configuration.SemanticVersionFormat,
             baseVersionSource: baseVersionSource,
             currentCommit: currentCommit,

src/GitVersion.Core/VersionCalculation/Mainline/NonTrunk/FirstCommitOnRelease.cs

@@ -30,7 +30,7 @@ public IEnumerable<IBaseVersionIncrement> GetIncrements(
         MainlineIteration iteration, MainlineCommit commit, MainlineContext context)
     {
         var effectiveConfiguration = commit.GetEffectiveConfiguration(context.Configuration);
-        if (!commit.BranchName.TryGetSemanticVersion(out var element, effectiveConfiguration)) yield break;
+        if (!commit.BranchName.TryGetSemanticVersion(effectiveConfiguration, out var element)) yield break;
         context.AlternativeSemanticVersions.Add(element.Value);
     }
 }

src/GitVersion.Core/VersionCalculation/Mainline/Trunk/CommitOnTrunkBranchedBase.cs

@@ -17,7 +17,7 @@ public virtual IEnumerable<IBaseVersionIncrement> GetIncrements(
 
         var effectiveConfiguration = iteration.GetEffectiveConfiguration(context.Configuration);
         if (iteration.GetEffectiveConfiguration(context.Configuration).IsReleaseBranch
-            && iteration.BranchName.TryGetSemanticVersion(out var element, effectiveConfiguration))
+            && iteration.BranchName.TryGetSemanticVersion(effectiveConfiguration, out var element))
         {
             context.AlternativeSemanticVersions.Add(element.Value);
         }

src/GitVersion.Core/VersionCalculation/VersionSearchStrategies/VersionInBranchNameVersionStrategy.cs

@@ -35,7 +35,7 @@ public bool TryGetBaseVersion(EffectiveBranchConfiguration configuration, [NotNu
 
         foreach (var branch in new[] { Context.CurrentBranch, configuration.Branch })
         {
-            if (!branch.Name.TryGetSemanticVersion(out var result, configuration.Value)) continue;
+            if (!branch.Name.TryGetSemanticVersion(configuration.Value, out var result)) continue;
             string? branchNameOverride = null;
             if (!result.Name.IsNullOrEmpty() && (Context.CurrentBranch.Name.Equals(branch.Name)
                                                  || Context.Configuration.GetBranchConfiguration(Context.CurrentBranch.Name).Label is null))