Skip to content

Commit fa26078

Browse files
committed
* ConfigurationProvider.cs refactoring: extracted WarnAboutAmbigousConfigFileSelection
1 parent 9d592ab commit fa26078

File tree

1 file changed

+31
-9
lines changed

1 file changed

+31
-9
lines changed

src/GitVersionCore/Configuration/ConfigurationProvider.cs

Lines changed: 31 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
namespace GitVersion
22
{
3+
using GitVersion.Configuration.Init.Wizard;
4+
using GitVersion.Helpers;
35
using System.Collections.Generic;
46
using System.ComponentModel;
57
using System.IO;
68
using System.Linq;
79
using System.Text;
8-
using GitVersion.Configuration.Init.Wizard;
9-
using GitVersion.Helpers;
1010

1111
public class ConfigurationProvider
1212
{
@@ -103,10 +103,27 @@ static void MigrateBranches(Config config)
103103
// Map of current names and previous names
104104
var dict = new Dictionary<string, string[]>
105105
{
106-
{ "hotfix(es)?[/-]", new [] { "hotfix[/-]" }},
107-
{ "features?[/-]", new [] { "feature[/-]", "feature(s)?[/-]" }},
108-
{ "releases?[/-]", new [] { "release[/-]" }},
109-
{ "dev(elop)?(ment)?$", new [] { "develop" }}
106+
{"hotfix(es)?[/-]", new[]
107+
{
108+
"hotfix[/-]"
109+
}
110+
},
111+
{"features?[/-]", new[]
112+
{
113+
"feature[/-]",
114+
"feature(s)?[/-]"
115+
}
116+
},
117+
{"releases?[/-]", new[]
118+
{
119+
"release[/-]"
120+
}
121+
},
122+
{"dev(elop)?(ment)?$", new[]
123+
{
124+
"develop"
125+
}
126+
}
110127
};
111128

112129
foreach (var mapping in dict)
@@ -186,12 +203,12 @@ public static void Verify(GitPreparer gitPreparer, IFileSystem fileSystem)
186203
var workingDirectory = gitPreparer.WorkingDirectory;
187204
var projectRootDirectory = gitPreparer.GetProjectRootDirectory();
188205

189-
Verify(workingDirectory,projectRootDirectory, fileSystem);
206+
Verify(workingDirectory, projectRootDirectory, fileSystem);
190207
}
191208

192209
public static void Verify(string workingDirectory, string projectRootDirectory, IFileSystem fileSystem)
193210
{
194-
if(fileSystem.PathsEqual(workingDirectory, projectRootDirectory))
211+
if (fileSystem.PathsEqual(workingDirectory, projectRootDirectory))
195212
{
196213
WarnAboutObsoleteConfigFile(workingDirectory, fileSystem);
197214
return;
@@ -200,6 +217,11 @@ public static void Verify(string workingDirectory, string projectRootDirectory,
200217
WarnAboutObsoleteConfigFile(workingDirectory, fileSystem);
201218
WarnAboutObsoleteConfigFile(projectRootDirectory, fileSystem);
202219

220+
WarnAboutAmbigousConfigFileSelection(workingDirectory, projectRootDirectory, fileSystem);
221+
}
222+
223+
private static void WarnAboutAmbigousConfigFileSelection(string workingDirectory, string projectRootDirectory, IFileSystem fileSystem)
224+
{
203225
var workingConfigFile = GetConfigFilePath(workingDirectory, fileSystem);
204226
var projectRootConfigFile = GetConfigFilePath(projectRootDirectory, fileSystem);
205227

@@ -280,4 +302,4 @@ public static void Init(string workingDirectory, IFileSystem fileSystem, IConsol
280302
}
281303
}
282304
}
283-
}
305+
}

0 commit comments

Comments
 (0)