Integrate code review remarks from arturcic

Author: HHobeck

docs/input/docs/reference/configuration.md

@@ -105,7 +105,7 @@ branches:
       of-merged-branch: true
       when-current-commit-tagged: false
     track-merge-target: false
-    regex: ^releases?[/-]
+    regex: ^releases?[/-](?<BranchName>.+)
     source-branches:
     - develop
     - main
@@ -153,7 +153,7 @@ branches:
     increment: Inherit
     prevent-increment:
       when-current-commit-tagged: false
-    regex: ^hotfix(es)?[/-]
+    regex: ^hotfix(es)?[/-](?<BranchName>.+)
     source-branches:
     - release
     - main
@@ -168,7 +168,7 @@ branches:
     prevent-increment:
       of-merged-branch: true
     track-merge-target: false
-    regex: ^support[/-]
+    regex: ^support[/-](?<BranchName>.+)
     source-branches:
     - main
     is-source-branch-for: []
@@ -209,7 +209,6 @@ is-source-branch-for: []
 tracks-release-branches: false
 is-release-branch: false
 is-main-branch: false
-
 ```
 
 The supported built-in configuration for the `GitHubFlow` workflow (`workflow: GitHubFlow/v1`) looks like:
@@ -257,7 +256,7 @@ branches:
       of-merged-branch: true
       when-current-commit-tagged: false
     track-merge-target: false
-    regex: ^releases?[/-]
+    regex: ^releases?[/-](?<BranchName>.+)
     source-branches:
     - main
     - release

src/GitVersion.Configuration.Tests/Configuration/ConfigurationProviderTests.CanWriteOutEffectiveConfiguration.approved.txt

@@ -54,7 +54,7 @@ branches:
       of-merged-branch: true
       when-current-commit-tagged: false
     track-merge-target: false
-    regex: ^releases?[/-]
+    regex: ^releases?[/-](?<BranchName>.+)
     source-branches:
     - develop
     - main
@@ -102,7 +102,7 @@ branches:
     increment: Inherit
     prevent-increment:
       when-current-commit-tagged: false
-    regex: ^hotfix(es)?[/-]
+    regex: ^hotfix(es)?[/-](?<BranchName>.+)
     source-branches:
     - release
     - main
@@ -117,7 +117,7 @@ branches:
     prevent-increment:
       of-merged-branch: true
     track-merge-target: false
-    regex: ^support[/-]
+    regex: ^support[/-](?<BranchName>.+)
     source-branches:
     - main
     is-source-branch-for: []

src/GitVersion.Configuration/SupportedWorkflows/GitFlow/v1.yml

@@ -54,7 +54,7 @@ branches:
       of-merged-branch: true
       when-current-commit-tagged: false
     track-merge-target: false
-    regex: ^releases?[/-]
+    regex: ^releases?[/-](?<BranchName>.+)
     source-branches:
     - develop
     - main
@@ -102,7 +102,7 @@ branches:
     increment: Inherit
     prevent-increment:
       when-current-commit-tagged: false
-    regex: ^hotfix(es)?[/-]
+    regex: ^hotfix(es)?[/-](?<BranchName>.+)
     source-branches:
     - release
     - main
@@ -117,7 +117,7 @@ branches:
     prevent-increment:
       of-merged-branch: true
     track-merge-target: false
-    regex: ^support[/-]
+    regex: ^support[/-](?<BranchName>.+)
     source-branches:
     - main
     is-source-branch-for: []

src/GitVersion.Configuration/SupportedWorkflows/GitHubFlow/v1.yml

@@ -40,7 +40,7 @@ branches:
       of-merged-branch: true
       when-current-commit-tagged: false
     track-merge-target: false
-    regex: ^releases?[/-]
+    regex: ^releases?[/-](?<BranchName>.+)
     source-branches:
     - main
     - release

src/GitVersion.Configuration/TrunkBasedConfigurationBuilder.cs

@@ -80,7 +80,7 @@ private TrunkBasedConfigurationBuilder()
         WithBranch(HotfixBranch.Name).WithConfiguration(new BranchConfiguration()
         {
             Increment = IncrementStrategy.Patch,
-            RegularExpression = "^hotfix(es)?[/-](?<BranchName>.+)",
+            RegularExpression = HotfixBranch.RegexPattern,
             SourceBranches =
             [
                 this.MainBranch.Name

src/GitVersion.Core.Tests/IntegrationTests/TrunkBasedDevelopmentScenarios.cs

@@ -36,27 +36,6 @@ private static GitFlowConfigurationBuilder GetConfigurationBuilder() => GitFlowC
             .WithSourceBranches("main")
         );
 
-    [Test]
-    [Ignore("Support of multiple tunks are not implemented at the moment.")]
-    public void VerifyNonMainMainlineVersionIdenticalAsMain()
-    {
-        var configuration = GetConfigurationBuilder().Build();
-
-        using var fixture = new EmptyRepositoryFixture();
-        fixture.Repository.MakeACommit("1");
-
-        fixture.BranchTo("feature/foo", "foo");
-        fixture.MakeACommit("2 +semver: major");
-        fixture.Checkout(MainBranch);
-        fixture.MergeNoFF("feature/foo");
-
-        fixture.AssertFullSemver("1.0.0", configuration);
-
-        fixture.BranchTo("support/1.0", "support");
-
-        fixture.AssertFullSemver("1.0.0", configuration);
-    }
-
     [Test]
     public void MergedFeatureBranchesToMainImpliesRelease()
     {

src/GitVersion.Core/Configuration/ConfigurationConstants.cs

@@ -42,10 +42,10 @@ internal static class ConfigurationConstants
 
     public const string MainBranchRegex = "^master$|^main$";
     public const string DevelopBranchRegex = "^dev(elop)?(ment)?$";
-    public const string ReleaseBranchRegex = "^releases?[/-]";
+    public const string ReleaseBranchRegex = "^releases?[/-](?<BranchName>.+)";
     public const string FeatureBranchRegex = "^features?[/-](?<BranchName>.+)";
     public const string PullRequestBranchRegex = @"^(pull|pull\-requests|pr)[/-]";
-    public const string HotfixBranchRegex = "^hotfix(es)?[/-]";
-    public const string SupportBranchRegex = "^support[/-]";
+    public const string HotfixBranchRegex = "^hotfix(es)?[/-](?<BranchName>.+)";
+    public const string SupportBranchRegex = "^support[/-](?<BranchName>.+)";
     public const string UnknownBranchRegex = "(?<BranchName>.+)";
 }

src/GitVersion.Core/MergeMessage.cs

@@ -15,7 +15,8 @@ public class MergeMessage
         new("BitBucketPullv7", @"^Pull request #(?<PullRequestNumber>\d+).*\r?\n\r?\nMerge in (?<Source>.*) from (?<SourceBranch>[^\s]*) to (?<TargetBranch>[^\s]*)"),
         new("BitBucketCloudPull", @"^Merged in (?<SourceBranch>[^\s]*) \(pull request #(?<PullRequestNumber>\d+)\)"),
         new("GitHubPull", @"^Merge pull request #(?<PullRequestNumber>\d+) (from|in) (?:[^\s\/]+\/)?(?<SourceBranch>[^\s]*)(?: into (?<TargetBranch>[^\s]*))*"),
-        new("RemoteTracking", @"^Merge remote-tracking branch '(?<SourceBranch>[^\s]*)'(?: into (?<TargetBranch>[^\s]*))*")
+        new("RemoteTracking", @"^Merge remote-tracking branch '(?<SourceBranch>[^\s]*)'(?: into (?<TargetBranch>[^\s]*))*"),
+        new("AzureDevOpsPull", @"^Merge pull request (?<PullRequestNumber>\d+) from (?<SourceBranch>[^\s]*) into (?<TargetBranch>[^\s]*)")
     };
 
     public MergeMessage(string mergeMessage, IGitVersionConfiguration configuration)