Merge branch 'master' into EOFRegex

asbjornu · web-flow · commit 9411d6f0278d · 2020-03-02T11:46:20.000+01:00
diff --git a/src/GitVersionCore.Tests/ConfigProviderTests.cs b/src/GitVersionCore.Tests/ConfigProviderTests.cs
@@ -96,7 +96,7 @@ public void RegexIsRequired()
         tag: bugfix";
             SetupConfigFileContent(text);
             var ex = Should.Throw<GitVersionConfigurationException>(() => configProvider.Provide(repoPath));
-            ex.Message.ShouldBe("Branch configuration 'bug' is missing required configuration 'regex'\n\n" +
+            ex.Message.ShouldBe($"Branch configuration 'bug' is missing required configuration 'regex'{System.Environment.NewLine}" +
                                 "See https://gitversion.net/docs/configuration/ for more info");
         }
 
@@ -111,7 +111,7 @@ public void SourceBranchIsRequired()
         tag: bugfix";
             SetupConfigFileContent(text);
             var ex = Should.Throw<GitVersionConfigurationException>(() => configProvider.Provide(repoPath));
-            ex.Message.ShouldBe("Branch configuration 'bug' is missing required configuration 'source-branches'\n\n" +
+            ex.Message.ShouldBe($"Branch configuration 'bug' is missing required configuration 'source-branches'{System.Environment.NewLine}" +
                                 "See https://gitversion.net/docs/configuration/ for more info");
         }
 
diff --git a/src/GitVersionCore.Tests/VariableProviderTests.cs b/src/GitVersionCore.Tests/VariableProviderTests.cs
@@ -43,7 +43,9 @@ public void ShouldLogWarningWhenUsingDefaultInformationalVersionInCustomFormat()
                 Patch = 3,
             };
 
+#pragma warning disable CS0618 // Type or member is obsolete
             var propertyName = nameof(SemanticVersionFormatValues.DefaultInformationalVersion);
+#pragma warning restore CS0618 // Type or member is obsolete
             var config = new TestEffectiveConfiguration(assemblyInformationalFormat: $"{{{propertyName}}}");
             variableProvider.GetVariablesFor(semVer, config, false);
             logMessages.ShouldContain(message => message.Trim().StartsWith("WARN") && message.Contains(propertyName), 1, $"Expected a warning to be logged when using the variable {propertyName} in a configuration format template");
diff --git a/src/GitVersionCore/BuildServers/TeamCity.cs b/src/GitVersionCore/BuildServers/TeamCity.cs
@@ -34,11 +34,8 @@ public override string GetCurrentBranch(bool usingDynamicRepos)
         private void WriteBranchEnvVariableWarning()
         {
             Log.Warning(@"TeamCity doesn't make the current branch available through environmental variables.
-
 Depending on your authentication and transport setup of your git VCS root things may work. In that case, ignore this warning.
-
 In your TeamCity build configuration, add a parameter called `env.Git_Branch` with value %teamcity.build.vcs.branch.<vcsid>%
-
 See https://gitversion.net/docs/build-server-support/build-server/teamcity for more info");
         }
 
diff --git a/src/GitVersionCore/Configuration/BranchConfigurationCalculator.cs b/src/GitVersionCore/Configuration/BranchConfigurationCalculator.cs
@@ -145,7 +145,7 @@ private BranchConfig InheritBranchConfiguration(Branch targetBranch, BranchConfi
                 }
 
                 var branchName = chosenBranch.FriendlyName;
-                log.Warning(errorMessage + System.Environment.NewLine + System.Environment.NewLine + "Falling back to " + branchName + " branch config");
+                log.Warning(errorMessage + System.Environment.NewLine + "Falling back to " + branchName + " branch config");
 
                 // To prevent infinite loops, make sure that a new branch was chosen.
                 if (targetBranch.IsSameBranch(chosenBranch))
diff --git a/src/GitVersionCore/Configuration/ConfigExtensions.cs b/src/GitVersionCore/Configuration/ConfigExtensions.cs
@@ -88,14 +88,14 @@ public static void Reset(this Config config)
                 var regex = branchConfig.Value.Regex;
                 if (regex == null)
                 {
-                    throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'regex'\n\n" +
+                    throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'regex'{System.Environment.NewLine}" +
                                                                "See https://gitversion.net/docs/configuration/ for more info");
                 }
 
                 var sourceBranches = branchConfig.Value.SourceBranches;
                 if (sourceBranches == null)
                 {
-                    throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'source-branches'\n\n" +
+                    throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'source-branches'{System.Environment.NewLine}" +
                                                                "See https://gitversion.net/docs/configuration/ for more info");
                 }
 
@@ -189,7 +189,7 @@ public static BranchConfig GetConfigForBranch(this Config config, string branchN
             }
             catch (InvalidOperationException)
             {
-                var matchingConfigs = String.Concat(matches.Select(m => $"\n - {m.Key}"));
+                var matchingConfigs = String.Concat(matches.Select(m => $"{System.Environment.NewLine} - {m.Key}"));
                 var picked = matches
                     .Select(kvp => kvp.Value)
                     .First();
diff --git a/src/GitVersionCore/Configuration/Init/SetConfig/ConfigureBranches.cs b/src/GitVersionCore/Configuration/Init/SetConfig/ConfigureBranches.cs
@@ -46,7 +46,7 @@ protected override string GetPrompt(Config config, string workingDirectory)
             return @"Which branch would you like to configure:
 
 0) Go Back
-" + string.Join("\r\n", OrderedBranches(config).Select((c, i) => $"{i + 1}) {c.Key}"));
+" + string.Join(System.Environment.NewLine, OrderedBranches(config).Select((c, i) => $"{i + 1}) {c.Key}"));
         }
 
         private static IOrderedEnumerable<KeyValuePair<string, BranchConfig>> OrderedBranches(Config config)
diff --git a/src/GitVersionCore/Configuration/Init/Wizard/FinishedSetupStep.cs b/src/GitVersionCore/Configuration/Init/Wizard/FinishedSetupStep.cs
@@ -10,7 +10,7 @@ public FinishedSetupStep(IConsole console, IFileSystem fileSystem, ILog log, ICo
 
         protected override string GetPrompt(Config config, string workingDirectory)
         {
-            return "Questions are all done, you can now edit GitVersion's configuration further\r\n" + base.GetPrompt(config, workingDirectory);
+            return $"Questions are all done, you can now edit GitVersion's configuration further{System.Environment.NewLine}" + base.GetPrompt(config, workingDirectory);
         }
     }
 }
diff --git a/src/GitVersionCore/Configuration/Init/Wizard/GitFlowSetupStep.cs b/src/GitVersionCore/Configuration/Init/Wizard/GitFlowSetupStep.cs
@@ -11,7 +11,7 @@ public GitFlowSetupStep(IConsole console, IFileSystem fileSystem, ILog log, ICon
 
         protected override string GetPrompt(Config config, string workingDirectory)
         {
-            return "By default GitVersion will only increment the version of the 'develop' branch every commit, all other branches will increment when tagged\r\n\r\n" +
+            return $"By default GitVersion will only increment the version of the 'develop' branch every commit, all other branches will increment when tagged{System.Environment.NewLine}{System.Environment.NewLine}" +
                 base.GetPrompt(config, workingDirectory);
         }
     }
diff --git a/src/GitVersionCore/Configuration/Init/Wizard/GitHubFlowStep.cs b/src/GitVersionCore/Configuration/Init/Wizard/GitHubFlowStep.cs
@@ -11,7 +11,7 @@ public GitHubFlowStep(IConsole console, IFileSystem fileSystem, ILog log, IConfi
 
         protected override string GetPrompt(Config config, string workingDirectory)
         {
-            return "By default GitVersion will only increment the version when tagged\r\n\r\n" + base.GetPrompt(config, workingDirectory);
+            return $"By default GitVersion will only increment the version when tagged{System.Environment.NewLine}{System.Environment.NewLine}" + base.GetPrompt(config, workingDirectory);
         }
     }
 }
diff --git a/src/GitVersionCore/GitRepoMetadataProvider.cs b/src/GitVersionCore/GitRepoMetadataProvider.cs
@@ -218,8 +218,8 @@ public BranchCommit FindCommitBranchWasBranchedFrom(Branch branch, params Branch
                 if (possibleBranches.Count > 1)
                 {
                     var first = possibleBranches.First();
-                    log.Info($"Multiple source branches have been found, picking the first one ({first.Branch.FriendlyName}).\n" +
-                        "This may result in incorrect commit counting.\nOptions were:\n " +
+                    log.Info($"Multiple source branches have been found, picking the first one ({first.Branch.FriendlyName}).{System.Environment.NewLine}" +
+                        $"This may result in incorrect commit counting.{System.Environment.NewLine}Options were:{System.Environment.NewLine}" +
                         string.Join(", ", possibleBranches.Select(b => b.Branch.FriendlyName)));
                     return first;
                 }
diff --git a/src/GitVersionCore/Helpers/GitRepositoryHelper.cs b/src/GitVersionCore/Helpers/GitRepositoryHelper.cs
@@ -62,7 +62,7 @@ public static void NormalizeGitDirectory(ILog log, IEnvironment environment, str
                 }
 
                 log.Info($"HEAD is detached and points at commit '{headSha}'.");
-                log.Info(string.Format("Local Refs:\r\n" + string.Join(System.Environment.NewLine, repo.Refs.FromGlob("*").Select(r => $"{r.CanonicalName} ({r.TargetIdentifier})"))));
+                log.Info($"Local Refs:{System.Environment.NewLine}" + string.Join(System.Environment.NewLine, repo.Refs.FromGlob("*").Select(r => $"{r.CanonicalName} ({r.TargetIdentifier})")));
 
                 // In order to decide whether a fake branch is required or not, first check to see if any local branches have the same commit SHA of the head SHA.
                 // If they do, go ahead and checkout that branch
@@ -203,7 +203,7 @@ private static void CreateFakeBranchPointingAtThePullRequestTip(ILog log, Reposi
                     GetRemoteTipsUsingUsernamePasswordCredentials(repo, remote, authentication.Username, authentication.Password))
                 .ToList();
 
-            log.Info("Remote Refs:\r\n" + string.Join(System.Environment.NewLine, remoteTips.Select(r => r.CanonicalName)));
+            log.Info($"Remote Refs:{System.Environment.NewLine}" + string.Join(System.Environment.NewLine, remoteTips.Select(r => r.CanonicalName)));
 
             var headTipSha = repo.Head.Tip.Sha;
 
diff --git a/src/GitVersionCore/Logging/FileAppender.cs b/src/GitVersionCore/Logging/FileAppender.cs
@@ -37,7 +37,7 @@ public void WriteTo(LogLevel level, string message)
 
         private static void WriteLogEntry(string logFilePath, string str)
         {
-            var contents = $"{DateTime.Now:yyyy-MM-dd HH:mm:ss}\t\t{str}\r\n";
+            var contents = $"{DateTime.Now:yyyy-MM-dd HH:mm:ss}\t\t{str}{System.Environment.NewLine}";
             File.AppendAllText(logFilePath, contents);
         }
     }
diff --git a/src/GitVersionCore/Logging/Log.cs b/src/GitVersionCore/Logging/Log.cs
@@ -41,7 +41,7 @@ public void Write(Verbosity verbosity, LogLevel level, string format, params obj
                 appender.WriteTo(level, formattedString);
             }
 
-            sb.Append(formattedString);
+            sb.AppendLine(formattedString);
         }
 
         public IDisposable IndentLog(string operationDescription)
diff --git a/src/GitVersionCore/OutputVariables/VariableProvider.cs b/src/GitVersionCore/OutputVariables/VariableProvider.cs
@@ -154,7 +154,9 @@ private string CheckAndFormatString<T>(string formatString, T source, IEnvironme
 
         private void WarnIfUsingObsoleteFormatValues(string formatString)
         {
+#pragma warning disable CS0618 // Type or member is obsolete
             var obsoletePropertyName = nameof(SemanticVersionFormatValues.DefaultInformationalVersion);
+#pragma warning restore CS0618 // Type or member is obsolete
             if (formatString.Contains($"{{{obsoletePropertyName}}}"))
             {
                 log.Write(LogLevel.Warn, $"Use format variable '{nameof(SemanticVersionFormatValues.InformationalVersion)}' instead of '{obsoletePropertyName}' which is obsolete and will be removed in a future release.");
diff --git a/src/GitVersionExe.Tests/ArgumentParserTests.cs b/src/GitVersionExe.Tests/ArgumentParserTests.cs
@@ -59,7 +59,9 @@ public void HMeansIsHelp()
         public void Exec()
         {
             var arguments = argumentParser.ParseArguments("-exec rake");
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Exec.ShouldBe("rake");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
@@ -72,15 +74,19 @@ public void ExecWithArgs()
                 "-execargs",
                 "clean build"
             });
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Exec.ShouldBe("rake");
             arguments.ExecArgs.ShouldBe("clean build");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
         public void Msbuild()
         {
             var arguments = argumentParser.ParseArguments("-proj msbuild.proj");
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Proj.ShouldBe("msbuild.proj");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
@@ -93,16 +99,20 @@ public void MsbuildWithArgs()
                 "-projargs",
                 "/p:Configuration=Debug /p:Platform=AnyCPU"
             });
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Proj.ShouldBe("msbuild.proj");
             arguments.ProjArgs.ShouldBe("/p:Configuration=Debug /p:Platform=AnyCPU");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
         public void ExecwithTargetdirectory()
         {
             var arguments = argumentParser.ParseArguments("targetDirectoryPath -exec rake");
             arguments.TargetPath.ShouldBe("targetDirectoryPath");
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Exec.ShouldBe("rake");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
@@ -374,23 +384,29 @@ public void OtherArgumentsCanBeParsedAfterNofetch()
         {
             var arguments = argumentParser.ParseArguments("-nofetch -proj foo.sln");
             arguments.NoFetch.ShouldBe(true);
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Proj.ShouldBe("foo.sln");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
         public void OtherArgumentsCanBeParsedAfterNonormalize()
         {
             var arguments = argumentParser.ParseArguments("-nonormalize -proj foo.sln");
             arguments.NoNormalize.ShouldBe(true);
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Proj.ShouldBe("foo.sln");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [Test]
         public void OtherArgumentsCanBeParsedAfterNocache()
         {
             var arguments = argumentParser.ParseArguments("-nocache -proj foo.sln");
             arguments.NoCache.ShouldBe(true);
+#pragma warning disable CS0612 // Type or member is obsolete
             arguments.Proj.ShouldBe("foo.sln");
+#pragma warning restore CS0612 // Type or member is obsolete
         }
 
         [TestCase("-nofetch -nonormalize -nocache")]
diff --git a/src/GitVersionExe/ArgumentParser.cs b/src/GitVersionExe/ArgumentParser.cs
@@ -149,29 +149,37 @@ public Arguments ParseArguments(string[] commandLineArguments)
                 if (name.IsSwitch("exec"))
                 {
                     EnsureArgumentValueCount(values);
+#pragma warning disable CS0612 // Type or member is obsolete
                     arguments.Exec = value;
+#pragma warning restore CS0612 // Type or member is obsolete
                     continue;
                 }
 
 
                 if (name.IsSwitch("execargs"))
                 {
                     EnsureArgumentValueCount(values);
+#pragma warning disable CS0612 // Type or member is obsolete
                     arguments.ExecArgs = value;
+#pragma warning restore CS0612 // Type or member is obsolete
                     continue;
                 }
 
                 if (name.IsSwitch("proj"))
                 {
                     EnsureArgumentValueCount(values);
+#pragma warning disable CS0612 // Type or member is obsolete
                     arguments.Proj = value;
+#pragma warning restore CS0612 // Type or member is obsolete
                     continue;
                 }
 
                 if (name.IsSwitch("projargs"))
                 {
                     EnsureArgumentValueCount(values);
+#pragma warning disable CS0612 // Type or member is obsolete
                     arguments.ProjArgs = value;
+#pragma warning restore CS0612 // Type or member is obsolete
                     continue;
                 }
 
@@ -238,8 +246,8 @@ public Arguments ParseArguments(string[] commandLineArguments)
 
                     if (versionVariable == null)
                     {
-                        var messageFormat = "{0} requires a valid version variable.  Available variables are:\n{1}";
-                        var message = string.Format(messageFormat, name, string.Join(", ", VersionVariables.AvailableVariables.Select(x => string.Concat("'", x, "'"))));
+                        var message = $"{name} requires a valid version variable. Available variables are:{System.Environment.NewLine}" +
+                            string.Join(", ", VersionVariables.AvailableVariables.Select(x => string.Concat("'", x, "'")));
                         throw new WarningException(message);
                     }
 
diff --git a/src/GitVersionExe/ExecCommand.cs b/src/GitVersionExe/ExecCommand.cs
@@ -83,16 +83,19 @@ public void Execute()
 
         private static bool RunMsBuildIfNeeded(Arguments args, string workingDirectory, VersionVariables variables, ILog log)
         {
+#pragma warning disable CS0612 // Type or member is obsolete
             if (string.IsNullOrEmpty(args.Proj)) return false;
 
             args.Exec = "dotnet";
             args.ExecArgs = $"msbuild \"{args.Proj}\" {args.ProjArgs}";
+#pragma warning restore CS0612 // Type or member is obsolete
 
             return RunExecCommandIfNeeded(args, workingDirectory, variables, log);
         }
 
         private static bool RunExecCommandIfNeeded(Arguments args, string workingDirectory, VersionVariables variables, ILog log)
         {
+#pragma warning disable CS0612 // Type or member is obsolete
             if (string.IsNullOrEmpty(args.Exec)) return false;
 
             log.Info($"Launching {args.Exec} {args.ExecArgs}");
@@ -103,6 +106,7 @@ private static bool RunExecCommandIfNeeded(Arguments args, string workingDirecto
 
             if (results != 0)
                 throw new WarningException($"Execution of {args.Exec} failed, non-zero return code");
+#pragma warning restore CS0612 // Type or member is obsolete
 
             return true;
         }
diff --git a/src/GitVersionExe/GitVersionExecutor.cs b/src/GitVersionExe/GitVersionExecutor.cs
@@ -76,7 +76,9 @@ private int VerifyArgumentsAndRun(Arguments arguments)
                     arguments.Output.Add(OutputType.BuildServer);
                 }
 
+#pragma warning disable CS0612 // Type or member is obsolete
                 if (!string.IsNullOrEmpty(arguments.Proj) || !string.IsNullOrEmpty(arguments.Exec))
+#pragma warning restore CS0612 // Type or member is obsolete
                 {
                     arguments.Output.Add(OutputType.BuildServer);
                 }
@@ -118,18 +120,17 @@ private int VerifyArgumentsAndRun(Arguments arguments)
             }
             catch (WarningException exception)
             {
-                var error = $"An error occurred:\r\n{exception.Message}";
+                var error = $"An error occurred:{System.Environment.NewLine}{exception.Message}";
                 log.Warning(error);
                 return 1;
             }
             catch (Exception exception)
             {
-                var error = $"An unexpected error occurred:\r\n{exception}";
+                var error = $"An unexpected error occurred:{System.Environment.NewLine}{exception}";
                 log.Error(error);
 
                 if (arguments == null) return 1;
 
-                log.Info(string.Empty);
                 log.Info("Attempting to show the current git graph (please include in issue): ");
                 log.Info("Showing max of 100 commits");
 
diff --git a/src/GitVersionTask.MsBuild/GitVersionTask.MsBuild.csproj b/src/GitVersionTask.MsBuild/GitVersionTask.MsBuild.csproj
@@ -8,7 +8,7 @@
 
     <ItemGroup>
         <PackageReference Include="System.Runtime.Loader" Version="4.3.0" />
-        <PackageReference Include="Microsoft.Build" Version="16.4.0" />
+        <PackageReference Include="Microsoft.Build" Version="16.4.0" NoWarn="NU1701" />
         <PackageReference Include="Microsoft.Build.Tasks.Core" Version="16.4.0" />
     </ItemGroup>
 
diff --git a/src/GitVersionTask/MsBuildAppender.cs b/src/GitVersionTask/MsBuildAppender.cs
@@ -29,7 +29,7 @@ public void WriteTo(LogLevel level, string message)
 
         private void WriteLogEntry(LogLevel level, string str)
         {
-            var contents = $"{DateTime.Now:yyyy-MM-dd HH:mm:ss}\t\t{str}\r\n";
+            var contents = $"{DateTime.Now:yyyy-MM-dd HH:mm:ss}\t\t{str}{System.Environment.NewLine}";
             switch (level)
             {
                 case LogLevel.None:

Original file line number	Diff line number	Diff line change
`@@ -34,11 +34,8 @@ public override string GetCurrentBranch(bool usingDynamicRepos)`
`34`	`34`	`private void WriteBranchEnvVariableWarning()`
`35`	`35`	`{`
`36`	`36`	`Log.Warning(@"TeamCity doesn't make the current branch available through environmental variables.`
`37`		`-`
`38`	`37`	`Depending on your authentication and transport setup of your git VCS root things may work. In that case, ignore this warning.`
`39`		`-`
`40`	`38`	In your TeamCity build configuration, add a parameter called `env.Git_Branch` with value %teamcity.build.vcs.branch.<vcsid>%
`41`		`-`
`42`	`39`	`See https://gitversion.net/docs/build-server-support/build-server/teamcity for more info");`
`43`	`40`	`}`
`44`	`41`
Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ private BranchConfig InheritBranchConfiguration(Branch targetBranch, BranchConfi`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`var branchName = chosenBranch.FriendlyName;`
`148`		`- log.Warning(errorMessage + System.Environment.NewLine + System.Environment.NewLine + "Falling back to " + branchName + " branch config");`
	`148`	`+ log.Warning(errorMessage + System.Environment.NewLine + "Falling back to " + branchName + " branch config");`
`149`	`149`
`150`	`150`	`// To prevent infinite loops, make sure that a new branch was chosen.`
`151`	`151`	`if (targetBranch.IsSameBranch(chosenBranch))`
Original file line number	Diff line number	Diff line change
`@@ -88,14 +88,14 @@ public static void Reset(this Config config)`
`88`	`88`	`var regex = branchConfig.Value.Regex;`
`89`	`89`	`if (regex == null)`
`90`	`90`	`{`
`91`		`- throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'regex'\n\n" +`
	`91`	`+ throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'regex'{System.Environment.NewLine}" +`
`92`	`92`	`"See https://gitversion.net/docs/configuration/ for more info");`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`var sourceBranches = branchConfig.Value.SourceBranches;`
`96`	`96`	`if (sourceBranches == null)`
`97`	`97`	`{`
`98`		`- throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'source-branches'\n\n" +`
	`98`	`+ throw new GitVersionConfigurationException($"Branch configuration '{branchConfig.Key}' is missing required configuration 'source-branches'{System.Environment.NewLine}" +`
`99`	`99`	`"See https://gitversion.net/docs/configuration/ for more info");`
`100`	`100`	`}`
`101`	`101`
`@@ -189,7 +189,7 @@ public static BranchConfig GetConfigForBranch(this Config config, string branchN`
`189`	`189`	`}`
`190`	`190`	`catch (InvalidOperationException)`
`191`	`191`	`{`
`192`		`- var matchingConfigs = String.Concat(matches.Select(m => $"\n - {m.Key}"));`
	`192`	`+ var matchingConfigs = String.Concat(matches.Select(m => $"{System.Environment.NewLine} - {m.Key}"));`
`193`	`193`	`var picked = matches`
`194`	`194`	`.Select(kvp => kvp.Value)`
`195`	`195`	`.First();`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ protected override string GetPrompt(Config config, string workingDirectory)`
`46`	`46`	`return @"Which branch would you like to configure:`
`47`	`47`
`48`	`48`	`0) Go Back`
`49`		`-" + string.Join("\r\n", OrderedBranches(config).Select((c, i) => $"{i + 1}) {c.Key}"));`
	`49`	`+" + string.Join(System.Environment.NewLine, OrderedBranches(config).Select((c, i) => $"{i + 1}) {c.Key}"));`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`private static IOrderedEnumerable<KeyValuePair<string, BranchConfig>> OrderedBranches(Config config)`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ public FinishedSetupStep(IConsole console, IFileSystem fileSystem, ILog log, ICo`
`10`	`10`
`11`	`11`	`protected override string GetPrompt(Config config, string workingDirectory)`
`12`	`12`	`{`
`13`		`- return "Questions are all done, you can now edit GitVersion's configuration further\r\n" + base.GetPrompt(config, workingDirectory);`
	`13`	`+ return $"Questions are all done, you can now edit GitVersion's configuration further{System.Environment.NewLine}" + base.GetPrompt(config, workingDirectory);`
`14`	`14`	`}`
`15`	`15`	`}`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ public GitFlowSetupStep(IConsole console, IFileSystem fileSystem, ILog log, ICon`
`11`	`11`
`12`	`12`	`protected override string GetPrompt(Config config, string workingDirectory)`
`13`	`13`	`{`
`14`		`- return "By default GitVersion will only increment the version of the 'develop' branch every commit, all other branches will increment when tagged\r\n\r\n" +`
	`14`	`+ return $"By default GitVersion will only increment the version of the 'develop' branch every commit, all other branches will increment when tagged{System.Environment.NewLine}{System.Environment.NewLine}" +`
`15`	`15`	`base.GetPrompt(config, workingDirectory);`
`16`	`16`	`}`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ public GitHubFlowStep(IConsole console, IFileSystem fileSystem, ILog log, IConfi`
`11`	`11`
`12`	`12`	`protected override string GetPrompt(Config config, string workingDirectory)`
`13`	`13`	`{`
`14`		`- return "By default GitVersion will only increment the version when tagged\r\n\r\n" + base.GetPrompt(config, workingDirectory);`
	`14`	`+ return $"By default GitVersion will only increment the version when tagged{System.Environment.NewLine}{System.Environment.NewLine}" + base.GetPrompt(config, workingDirectory);`
`15`	`15`	`}`
`16`	`16`	`}`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -218,8 +218,8 @@ public BranchCommit FindCommitBranchWasBranchedFrom(Branch branch, params Branch`
`218`	`218`	`if (possibleBranches.Count > 1)`
`219`	`219`	`{`
`220`	`220`	`var first = possibleBranches.First();`
`221`		`- log.Info($"Multiple source branches have been found, picking the first one ({first.Branch.FriendlyName}).\n" +`
`222`		`- "This may result in incorrect commit counting.\nOptions were:\n " +`
	`221`	`+ log.Info($"Multiple source branches have been found, picking the first one ({first.Branch.FriendlyName}).{System.Environment.NewLine}" +`
	`222`	`+ $"This may result in incorrect commit counting.{System.Environment.NewLine}Options were:{System.Environment.NewLine}" +`
`223`	`223`	`string.Join(", ", possibleBranches.Select(b => b.Branch.FriendlyName)));`
`224`	`224`	`return first;`
`225`	`225`	`}`