Fixed #659 by adjusting the regexp

mkoertgen · mkoertgen · commit 9e30c40fbc95 · 2015-10-04T15:01:13.000+02:00
- Added negative lookbehind to filter out version string prefixed with '://' like 'file|http[s]://' etc.
diff --git a/src/GitVersionCore/VersionCalculation/BaseVersionCalculators/MergeMessageBaseVersionStrategy.cs b/src/GitVersionCore/VersionCalculation/BaseVersionCalculators/MergeMessageBaseVersionStrategy.cs
@@ -42,7 +42,7 @@ private static SemanticVersion Inner(Commit mergeCommit, EffectiveConfiguration
             }
 
             //TODO: Make the version prefixes customizable
-            var possibleVersions = Regex.Matches(mergeCommit.Message, @"^.*?(([rR]elease|[hH]otfix|[aA]lpha)-|-v|/|/v|'|Finish )(?<PossibleVersions>\d+\.\d+(\.*\d+)*)")
+            var possibleVersions = Regex.Matches(mergeCommit.Message, @"^.*?(([rR]elease|[hH]otfix|[aA]lpha)-|-v|/|/v|'|Finish )(?<PossibleVersions>(?<!://)\d+\.\d+(\.*\d+)*)")
                 .Cast<Match>()
                 .Select(m => m.Groups["PossibleVersions"].Value);
 

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ private static SemanticVersion Inner(Commit mergeCommit, EffectiveConfiguration`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`//TODO: Make the version prefixes customizable`
`45`		`- var possibleVersions = Regex.Matches(mergeCommit.Message, @"^.?(([rR]elease\|[hH]otfix\|[aA]lpha)-\|-v\|/\|/v\|'\|Finish )(?<PossibleVersions>\d+\.\d+(\.\d+)*)")`
	`45`	`+ var possibleVersions = Regex.Matches(mergeCommit.Message, @"^.?(([rR]elease\|[hH]otfix\|[aA]lpha)-\|-v\|/\|/v\|'\|Finish )(?<PossibleVersions>(?<!://)\d+\.\d+(\.\d+)*)")`
`46`	`46`	`.Cast<Match>()`
`47`	`47`	`.Select(m => m.Groups["PossibleVersions"].Value);`
`48`	`48`