Skip to content

Commit 7655c50

Browse files
committed
GitTools#2340 undo coding changes
there might be an easier solution
1 parent b9430d7 commit 7655c50

13 files changed

+29
-43
lines changed

src/GitVersionCore/Configuration/ConfigExtensions.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,10 +126,7 @@ public static string GetBranchSpecificTag(this EffectiveConfiguration configurat
126126
}
127127
if (tagToUse.Contains("{BranchName}"))
128128
{
129-
if (log != null)
130-
{
131-
log.Info("Using branch name to calculate version tag");
132-
}
129+
log.Info("Using branch name to calculate version tag");
133130

134131
var branchName = branchNameOverride ?? branchFriendlyName;
135132
if (!string.IsNullOrWhiteSpace(configuration.BranchPrefixToTrim))

src/GitVersionCore/Configuration/Init/SetNextVersion.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ protected override StepResult HandleResult(string result, Queue<ConfigInitWizard
2020
return StepResult.Ok();
2121
}
2222

23-
if (!SemanticVersion.TryParse(result, string.Empty, null, out var semVer))
23+
if (!SemanticVersion.TryParse(result, string.Empty, out var semVer))
2424
return StepResult.InvalidResponseSelected();
2525

2626
config.NextVersion = semVer.ToString("t");

src/GitVersionCore/Core/Abstractions/IRepositoryMetadataProvider.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,10 @@ public interface IRepositoryMetadataProvider
3434
/// </summary>
3535
BranchCommit FindCommitBranchWasBranchedFrom(Branch branch, Config configuration, params Branch[] excludedBranches);
3636

37-
SemanticVersion GetCurrentCommitTaggedVersion(Branch branch, GitObject commit, EffectiveConfiguration config);
37+
SemanticVersion GetCurrentCommitTaggedVersion(GitObject commit, EffectiveConfiguration config);
3838
SemanticVersion MaybeIncrement(BaseVersion baseVersion, GitVersionContext context);
39-
IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string tagPrefixRegex, string tagSuffix);
40-
IEnumerable<Tuple<Tag, SemanticVersion>> GetValidVersionTags(string tagPrefixRegex, string tagSuffix, DateTimeOffset? olderThan = null);
39+
IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string tagPrefixRegex);
40+
IEnumerable<Tuple<Tag, SemanticVersion>> GetValidVersionTags(string tagPrefixRegex, DateTimeOffset? olderThan = null);
4141

4242
ICommitLog GetCommitLog(Commit baseVersionSource, Commit currentCommit);
4343
bool GetMatchingCommitBranch(Commit baseVersionSource, Branch branch, Commit firstMatchingCommit);

src/GitVersionCore/Core/GitVersionContextFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ private GitVersionContext Init(Branch currentBranch, string commitId = null, boo
4545

4646
var currentBranchConfig = branchConfigurationCalculator.GetBranchConfiguration(currentBranch, currentCommit, configuration);
4747
var effectiveConfiguration = configuration.CalculateEffectiveConfiguration(currentBranchConfig);
48-
var currentCommitTaggedVersion = repositoryMetadataProvider.GetCurrentCommitTaggedVersion(currentBranch, currentCommit, effectiveConfiguration);
48+
var currentCommitTaggedVersion = repositoryMetadataProvider.GetCurrentCommitTaggedVersion(currentCommit, effectiveConfiguration);
4949
var numberOfUncommittedChanges = repositoryMetadataProvider.GetNumberOfUncommittedChanges();
5050

5151
return new GitVersionContext(currentBranch, currentCommit, configuration, effectiveConfiguration, currentCommitTaggedVersion, numberOfUncommittedChanges);

src/GitVersionCore/Core/RepositoryMetadataProvider.cs

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -348,13 +348,12 @@ public BranchCommit FindCommitBranchWasBranchedFrom(Branch branch, Config config
348348
}
349349

350350

351-
public SemanticVersion GetCurrentCommitTaggedVersion(Branch branch, GitObject commit, EffectiveConfiguration config)
351+
public SemanticVersion GetCurrentCommitTaggedVersion(GitObject commit, EffectiveConfiguration config)
352352
{
353353
return repository.Tags
354354
.SelectMany(t =>
355355
{
356-
var tag = config.GetBranchSpecificTag(null, branch.FriendlyName, null);
357-
if (t.PeeledTarget() == commit && SemanticVersion.TryParse(t.FriendlyName, config.GitTagPrefix, tag, out var version))
356+
if (t.PeeledTarget() == commit && SemanticVersion.TryParse(t.FriendlyName, config.GitTagPrefix, out var version))
358357
return new[] {
359358
version
360359
};
@@ -369,7 +368,7 @@ public SemanticVersion MaybeIncrement(BaseVersion baseVersion, GitVersionContext
369368
return increment != null ? baseVersion.SemanticVersion.IncrementVersion(increment.Value) : baseVersion.SemanticVersion;
370369
}
371370

372-
public IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string tagPrefixRegex, string tagSuffix)
371+
public IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string tagPrefixRegex)
373372
{
374373
if (semanticVersionTagsOnBranchCache.ContainsKey(branch))
375374
{
@@ -379,7 +378,7 @@ public IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string
379378

380379
using (log.IndentLog($"Getting version tags from branch '{branch.CanonicalName}'."))
381380
{
382-
var tags = GetValidVersionTags(tagPrefixRegex, tagSuffix);
381+
var tags = GetValidVersionTags(tagPrefixRegex);
383382

384383
var versionTags = branch.Commits.SelectMany(c => tags.Where(t => c.Sha == t.Item1.Target.Sha).Select(t => t.Item2)).ToList();
385384

@@ -388,7 +387,7 @@ public IEnumerable<SemanticVersion> GetVersionTagsOnBranch(Branch branch, string
388387
}
389388
}
390389

391-
public IEnumerable<Tuple<Tag, SemanticVersion>> GetValidVersionTags(string tagPrefixRegex, string tagSuffix, DateTimeOffset? olderThan = null)
390+
public IEnumerable<Tuple<Tag, SemanticVersion>> GetValidVersionTags(string tagPrefixRegex, DateTimeOffset? olderThan = null)
392391
{
393392
var tags = new List<Tuple<Tag, SemanticVersion>>();
394393

@@ -397,7 +396,7 @@ public IEnumerable<Tuple<Tag, SemanticVersion>> GetValidVersionTags(string tagPr
397396
if (!(tag.PeeledTarget() is Commit commit) || (olderThan.HasValue && commit.When() > olderThan.Value))
398397
continue;
399398

400-
if (SemanticVersion.TryParse(tag.FriendlyName, tagPrefixRegex, tagSuffix, out var semver))
399+
if (SemanticVersion.TryParse(tag.FriendlyName, tagPrefixRegex, out var semver))
401400
{
402401
tags.Add(Tuple.Create(tag, semver));
403402
}

src/GitVersionCore/Model/MergeMessage.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public MergeMessage(string mergeMessage, Config config)
4747
PullRequestNumber = pullNumber;
4848
}
4949

50-
Version = ParseVersion(config.TagPrefix, null);
50+
Version = ParseVersion(config.TagPrefix);
5151

5252
break;
5353
}
@@ -61,7 +61,7 @@ public MergeMessage(string mergeMessage, Config config)
6161
public int? PullRequestNumber { get; }
6262
public SemanticVersion Version { get; }
6363

64-
private SemanticVersion ParseVersion(string tagPrefix, string tagSuffix)
64+
private SemanticVersion ParseVersion(string tagPrefix)
6565
{
6666
// Remove remotes and branch prefixes like release/ feature/ hotfix/ etc
6767
var toMatch = Regex.Replace(MergedBranch, @"^(\w+[-/])*", "", RegexOptions.IgnoreCase);
@@ -70,7 +70,7 @@ private SemanticVersion ParseVersion(string tagPrefix, string tagSuffix)
7070
var versionMatch = new Regex(@"^(?<!://)\d+\.\d+(\.*\d+)*");
7171
var version = versionMatch.Match(toMatch);
7272

73-
if (version.Success && SemanticVersion.TryParse(version.Value, tagPrefix, tagSuffix, out var val))
73+
if (version.Success && SemanticVersion.TryParse(version.Value, tagPrefix, out var val))
7474
{
7575
return val;
7676
}

src/GitVersionCore/VersionCalculation/BaseVersionCalculators/ConfigNextVersionVersionStrategy.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
using System;
22
using System.Collections.Generic;
3-
using GitVersion.Common;
4-
using GitVersion.Configuration;
5-
using GitVersion.Extensions;
63

74
namespace GitVersion.VersionCalculation
85
{
@@ -21,7 +18,7 @@ public override IEnumerable<BaseVersion> GetVersions()
2118
{
2219
if (string.IsNullOrEmpty(Context.Configuration.NextVersion) || Context.IsCurrentCommitTagged)
2320
yield break;
24-
var semanticVersion = SemanticVersion.Parse(Context.Configuration.NextVersion, Context.Configuration.GitTagPrefix, null);
21+
var semanticVersion = SemanticVersion.Parse(Context.Configuration.NextVersion, Context.Configuration.GitTagPrefix);
2522
yield return new BaseVersion("NextVersion in GitVersion configuration file", false, semanticVersion, null, null);
2623
}
2724
}

src/GitVersionCore/VersionCalculation/BaseVersionCalculators/TaggedCommitVersionStrategy.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,7 @@ public override IEnumerable<BaseVersion> GetVersions()
2828

2929
internal IEnumerable<BaseVersion> GetTaggedVersions(Branch currentBranch, DateTimeOffset? olderThan)
3030
{
31-
var tagToUse = "";//Context.Configuration.GetBranchSpecificTag(null, currentBranch.FriendlyName, null);
32-
var allTags = repositoryMetadataProvider.GetValidVersionTags(Context.Configuration.GitTagPrefix, tagToUse, olderThan);
31+
var allTags = repositoryMetadataProvider.GetValidVersionTags(Context.Configuration.GitTagPrefix, olderThan);
3332

3433

3534
var taggedVersions = currentBranch

src/GitVersionCore/VersionCalculation/BaseVersionCalculators/TrackReleaseBranchesVersionStrategy.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@ private IEnumerable<BaseVersion> ReleaseBranchBaseVersions()
8484
private IEnumerable<BaseVersion> GetReleaseVersion(GitVersionContext context, Branch releaseBranch)
8585
{
8686
var tagPrefixRegex = context.Configuration.GitTagPrefix;
87-
var tagSuffix = Context.Configuration.Tag;
8887

8988
// Find the commit where the child branch was created.
9089
var baseSource = repositoryMetadataProvider.FindMergeBase(releaseBranch, context.CurrentBranch);
@@ -95,7 +94,7 @@ private IEnumerable<BaseVersion> GetReleaseVersion(GitVersionContext context, Br
9594
}
9695

9796
return releaseVersionStrategy
98-
.GetVersions(tagPrefixRegex, tagSuffix, releaseBranch)
97+
.GetVersions(tagPrefixRegex, releaseBranch)
9998
.Select(b => new BaseVersion(b.Source, true, b.SemanticVersion, baseSource, b.BranchNameOverride));
10099
}
101100
}

src/GitVersionCore/VersionCalculation/BaseVersionCalculators/VersionInBranchNameVersionStrategy.cs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,18 @@ public override IEnumerable<BaseVersion> GetVersions()
2525
{
2626
var currentBranch = Context.CurrentBranch;
2727
var tagPrefixRegex = Context.Configuration.GitTagPrefix;
28-
var tagSuffix = Context.Configuration.GetBranchSpecificTag(null, currentBranch.FriendlyName, null);
29-
return GetVersions(tagPrefixRegex, tagSuffix, currentBranch);
28+
return GetVersions(tagPrefixRegex, currentBranch);
3029
}
3130

32-
internal IEnumerable<BaseVersion> GetVersions(string tagPrefixRegex, string tagSuffix, Branch currentBranch)
31+
internal IEnumerable<BaseVersion> GetVersions(string tagPrefixRegex Branch currentBranch)
3332
{
3433
if (!Context.FullConfiguration.IsReleaseBranch(currentBranch.NameWithoutOrigin()))
3534
{
3635
yield break;
3736
}
3837

3938
var branchName = currentBranch.FriendlyName;
40-
var versionInBranch = GetVersionInBranch(branchName, tagPrefixRegex, tagSuffix);
39+
var versionInBranch = GetVersionInBranch(branchName, tagPrefixRegex);
4140
if (versionInBranch != null)
4241
{
4342
var commitBranchWasBranchedFrom = repositoryMetadataProvider.FindCommitBranchWasBranchedFrom(currentBranch, Context.FullConfiguration);
@@ -46,12 +45,12 @@ internal IEnumerable<BaseVersion> GetVersions(string tagPrefixRegex, string tagS
4645
}
4746
}
4847

49-
private Tuple<string, SemanticVersion> GetVersionInBranch(string branchName, string tagPrefixRegex, string tagSuffix)
48+
private Tuple<string, SemanticVersion> GetVersionInBranch(string branchName, string tagPrefixRegex)
5049
{
5150
var branchParts = branchName.Split('/', '-');
5251
foreach (var part in branchParts)
5352
{
54-
if (SemanticVersion.TryParse(part, tagPrefixRegex, tagSuffix, out var semanticVersion))
53+
if (SemanticVersion.TryParse(part, tagPrefixRegex, out var semanticVersion))
5554
{
5655
return Tuple.Create(part, semanticVersion);
5756
}

src/GitVersionCore/VersionCalculation/NextVersionCalculator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ private void UpdatePreReleaseTag(SemanticVersion semanticVersion, string branchN
111111
int? number = null;
112112

113113
var lastTag = repositoryMetadataProvider
114-
.GetVersionTagsOnBranch(context.CurrentBranch, context.Configuration.GitTagPrefix, tagToUse)
114+
.GetVersionTagsOnBranch(context.CurrentBranch, context.Configuration.GitTagPrefix)
115115
.FirstOrDefault(v => v.PreReleaseTag.Name.IsEquivalentTo(tagToUse));
116116

117117
if (lastTag != null &&

src/GitVersionCore/VersionCalculation/SemanticVersioning/SemanticVersion.cs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -136,21 +136,17 @@ public override int GetHashCode()
136136
return v1.CompareTo(v2) < 0;
137137
}
138138

139-
public static SemanticVersion Parse(string version, string tagPrefixRegex, string tagSuffix)
139+
public static SemanticVersion Parse(string version, string tagPrefixRegex)
140140
{
141-
if (!TryParse(version, tagPrefixRegex, tagSuffix, out var semanticVersion))
141+
if (!TryParse(version, tagPrefixRegex, out var semanticVersion))
142142
throw new WarningException($"Failed to parse {version} into a Semantic Version");
143143

144144
return semanticVersion;
145145
}
146146

147-
public static bool TryParse(string version, string tagPrefixRegex, string tagSuffix, out SemanticVersion semanticVersion)
147+
public static bool TryParse(string version, string tagPrefixRegex, out SemanticVersion semanticVersion)
148148
{
149-
if (tagSuffix == null)
150-
{
151-
tagSuffix = ".*";
152-
}
153-
var match = Regex.Match(version, $"^({tagPrefixRegex})?(?<version>[0-9.]+-?{tagSuffix}[.]?\\d*)$");
149+
var match = Regex.Match(version, $"^({tagPrefixRegex})?(?<version>.*)$");
154150

155151
if (!match.Success)
156152
{

src/GitVersionCore/VersionCalculation/SemanticVersioning/SemanticVersionExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ public static class SemanticVersionExtensions
66
{
77
public static void OverrideVersionManuallyIfNeeded(this SemanticVersion version, EffectiveConfiguration configuration)
88
{
9-
if (!string.IsNullOrEmpty(configuration.NextVersion) && SemanticVersion.TryParse(configuration.NextVersion, configuration.GitTagPrefix, null, out var manualNextVersion))
9+
if (!string.IsNullOrEmpty(configuration.NextVersion) && SemanticVersion.TryParse(configuration.NextVersion, configuration.GitTagPrefix, out var manualNextVersion))
1010
{
1111
if (manualNextVersion > version)
1212
{

0 commit comments

Comments
 (0)