Errors should be warnings

distantcam · distantcam · commit 53945c599857 · 2014-06-10T12:26:27.000+08:00
diff --git a/GitVersionCore/ArgumentParser.cs b/GitVersionCore/ArgumentParser.cs
@@ -147,14 +147,14 @@ public static Arguments ParseArguments(List<string> commandLineArguments)
                     OutputType outputType;
                     if (!Enum.TryParse(value, true, out outputType))
                     {
-                        throw new ErrorException(string.Format("Value '{0}' cannot be parsed as output type, please use 'json' or 'buildserver'", value));
+                        throw new WarningException(string.Format("Value '{0}' cannot be parsed as output type, please use 'json' or 'buildserver'", value));
                     }
 
                     arguments.Output = outputType;
                     continue;
                 }
 
-                throw new ErrorException(string.Format("Could not parse command line parameter '{0}'.", name));
+                throw new WarningException(string.Format("Could not parse command line parameter '{0}'.", name));
             }
             return arguments;
         }
diff --git a/GitVersionCore/BuildServers/AppVeyor.cs b/GitVersionCore/BuildServers/AppVeyor.cs
@@ -22,7 +22,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)
         {
             if (string.IsNullOrEmpty(gitDirectory))
             {
-                throw new ErrorException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");
+                throw new WarningException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");
             }
 
             var repoBranch = Environment.GetEnvironmentVariable("APPVEYOR_REPO_BRANCH");
diff --git a/GitVersionCore/BuildServers/ContinuaCi.cs b/GitVersionCore/BuildServers/ContinuaCi.cs
@@ -32,7 +32,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)
         {
             if (string.IsNullOrEmpty(gitDirectory))
             {
-                throw new ErrorException("Failed to find .git directory on agent");
+                throw new WarningException("Failed to find .git directory on agent");
             }
 
             GitHelper.NormalizeGitDirectory(gitDirectory, arguments);
diff --git a/GitVersionCore/BuildServers/GitHelper.cs b/GitVersionCore/BuildServers/GitHelper.cs
@@ -86,14 +86,14 @@ static void CreateFakeBranchPointingAtThePullRequestTip(Repository repo, Argumen
             if (refs.Count == 0)
             {
                 var message = string.Format("Couldn't find any remote tips from remote '{0}' pointing at the commit '{1}'.", remote.Url, headTipSha);
-                throw new ErrorException(message);
+                throw new WarningException(message);
             }
 
             if (refs.Count > 1)
             {
                 var names = string.Join(", ", refs.Select(r => r.CanonicalName));
                 var message = string.Format("Found more than one remote tip from remote '{0}' pointing at the commit '{1}'. Unable to determine which one to use ({2}).", remote.Url, headTipSha, names);
-                throw new ErrorException(message);
+                throw new WarningException(message);
             }
 
             var canonicalName = refs[0].CanonicalName;
@@ -102,7 +102,7 @@ static void CreateFakeBranchPointingAtThePullRequestTip(Repository repo, Argumen
             if (!canonicalName.StartsWith("refs/pull/"))
             {
                 var message = string.Format("Remote tip '{0}' from remote '{1}' doesn't look like a valid pull request.", canonicalName, remote.Url);
-                throw new ErrorException(message);
+                throw new WarningException(message);
             }
 
             var fakeBranchName = canonicalName.Replace("refs/pull/", "refs/heads/pull/");
@@ -168,7 +168,7 @@ static Remote EnsureOnlyOneRemoteIsDefined(IRepository repo)
             }
 
             var message = string.Format("{0} remote(s) have been detected. When being run on a TeamCity agent, the Git repository is expected to bear one (and no more than one) remote.", howMany);
-            throw new ErrorException(message);
+            throw new WarningException(message);
         }
     }
 }
diff --git a/GitVersionCore/BuildServers/TeamCity.cs b/GitVersionCore/BuildServers/TeamCity.cs
@@ -20,7 +20,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)
         {
             if (string.IsNullOrEmpty(gitDirectory))
             {
-                throw new ErrorException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");
+                throw new WarningException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");
             }
 
             GitHelper.NormalizeGitDirectory(gitDirectory, arguments);
diff --git a/GitVersionCore/GitFlow/BranchFinders/DevelopBasedVersionFinderBase.cs b/GitVersionCore/GitFlow/BranchFinders/DevelopBasedVersionFinderBase.cs
@@ -65,7 +65,7 @@ Commit FindCommonAncestorWithDevelop(IRepository repo, Branch branch, BranchType
                 return ancestor;
             }
 
-            throw new ErrorException(
+            throw new WarningException(
                 string.Format("A {0} branch is expected to branch off of 'develop'. "
                               + "However, branch 'develop' and '{1}' do not share a common ancestor."
                     , branchType, branch.Name));
diff --git a/GitVersionCore/GitFlow/BranchFinders/MasterVersionFinder.cs b/GitVersionCore/GitFlow/BranchFinders/MasterVersionFinder.cs
@@ -26,7 +26,7 @@ public SemanticVersion FindVersion(IRepository repository, Commit tip)
                 }
             }
 
-            throw new ErrorException("The head of master should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");
+            throw new WarningException("The head of master should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");
         }
 
         SemanticVersion BuildVersion(IRepository repository, Commit tip, int major, int minor, int patch)
diff --git a/GitVersionCore/GitFlow/BranchFinders/OptionallyTaggedBranchVersionFinderBase.cs b/GitVersionCore/GitFlow/BranchFinders/OptionallyTaggedBranchVersionFinderBase.cs
@@ -105,23 +105,23 @@ void EnsureVersionIsValid(SemanticVersion version, Branch branch, BranchType bra
 
             if (version.PreReleaseTag.HasTag())
             {
-                throw new ErrorException(msg + string.Format("Supported format is '{0}-Major.Minor.Patch'.", branchType.ToString().ToLowerInvariant()));
+                throw new WarningException(msg + string.Format("Supported format is '{0}-Major.Minor.Patch'.", branchType.ToString().ToLowerInvariant()));
             }
 
             switch (branchType)
             {
                 case BranchType.Hotfix:
                     if (version.Patch == 0)
                     {
-                        throw new ErrorException(msg + "A patch segment different than zero is required.");
+                        throw new WarningException(msg + "A patch segment different than zero is required.");
                     }
 
                     break;
 
                 case BranchType.Release:
                     if (version.Patch != 0)
                     {
-                        throw new ErrorException(msg + "A patch segment equals to zero is required.");
+                        throw new WarningException(msg + "A patch segment equals to zero is required.");
                     }
 
                     break;
@@ -154,7 +154,7 @@ int NumberOfCommitsInBranchNotKnownFromBaseBranch(
             if (ancestor == null)
             {
                 var message = string.Format("A {0} branch is expected to branch off of '{1}'. However, branch '{1}' and '{2}' do not share a common ancestor.", branchType, baseBranchName, branch.Name);
-                throw new ErrorException(message);
+                throw new WarningException(message);
             }
 
             var filter = new CommitFilter
diff --git a/GitVersionCore/GitFlow/BranchFinders/PullVersionFinder.cs b/GitVersionCore/GitFlow/BranchFinders/PullVersionFinder.cs
@@ -31,7 +31,7 @@ string ExtractIssueNumber(GitVersionContext context)
 
             if (!LooksLikeAValidPullRequestNumber(issueNumber))
             {
-                throw new ErrorException(string.Format("Unable to extract pull request number from '{0}'.",
+                throw new WarningException(string.Format("Unable to extract pull request number from '{0}'.",
                     pullRequestBranch.CanonicalName));
             }
 
diff --git a/GitVersionCore/GitFlow/BranchFinders/SupportVersionFinder.cs b/GitVersionCore/GitFlow/BranchFinders/SupportVersionFinder.cs
@@ -26,7 +26,7 @@ public SemanticVersion FindVersion(IRepository repository, Commit tip)
                 }
             }
 
-            throw new ErrorException("The head of a support branch should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");
+            throw new WarningException("The head of a support branch should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");
         }
 
         SemanticVersion BuildVersion(IRepository repository, Commit tip, int major, int minor, int patch)
diff --git a/GitVersionCore/GitHubFlow/BuildNumberCalculator.cs b/GitVersionCore/GitHubFlow/BuildNumberCalculator.cs
@@ -87,7 +87,7 @@ string ExtractIssueNumber(GitVersionContext context)
             if (!LooksLikeAValidPullRequestNumber(issueNumber))
             {
                 var message = string.Format("Unable to extract pull request number from '{0}'.", pullRequestBranch.CanonicalName);
-                throw new ErrorException(message);
+                throw new WarningException(message);
             }
 
             return issueNumber;
diff --git a/GitVersionCore/GitVersionContext.cs b/GitVersionCore/GitVersionContext.cs
@@ -16,7 +16,11 @@ public GitVersionContext(IRepository repository, Branch currentBranch)
         {
             Repository = repository;
             CurrentBranch = currentBranch;
-            CurrentCommit = CurrentBranch.Tip;
+
+            if (CurrentBranch != null)
+            {
+                CurrentCommit = CurrentBranch.Tip;
+            }
         }
 
         public IRepository Repository { get; private set; }
diff --git a/GitVersionCore/GitVersionCore.csproj b/GitVersionCore/GitVersionCore.csproj
@@ -67,7 +67,7 @@
     <Compile Include="BuildServers\TeamCity.cs" />
     <Compile Include="CachedVersion.cs" />
     <Compile Include="DirectoryDateFinder.cs" />
-    <Compile Include="ErrorException.cs" />
+    <Compile Include="WarningException.cs" />
     <Compile Include="ExtensionMethods.cs" />
     <Compile Include="GitDirFinder.cs" />
     <Compile Include="GitFlow\BranchClassifier.cs" />
diff --git a/GitVersionCore/GitVersionFinder.cs b/GitVersionCore/GitVersionFinder.cs
@@ -40,7 +40,7 @@ void EnsureHeadIsNotDetached(GitVersionContext context)
             }
 
             var message = string.Format("It looks like the branch being examined is a detached Head pointing to commit '{0}'. Without a proper branch name GitVersion cannot determine the build version.", context.CurrentCommit.Id.ToString(7));
-            throw new ErrorException(message);
+            throw new WarningException(message);
         }
 
         void EnsureLocalBranchExists(IRepository repository, string branchName)
@@ -51,7 +51,7 @@ void EnsureLocalBranchExists(IRepository repository, string branchName)
             }
 
             var existingBranches = string.Format("'{0}'", string.Join("', '", repository.Branches.Select(x => x.CanonicalName)));
-            throw new ErrorException(string.Format("This repository doesn't contain a branch named '{0}'. Please create one. Existing branches: {1}", branchName, existingBranches));
+            throw new WarningException(string.Format("This repository doesn't contain a branch named '{0}'. Please create one. Existing branches: {1}", branchName, existingBranches));
         }
     }
 }
diff --git a/GitVersionCore/RepositoryLoader.cs b/GitVersionCore/RepositoryLoader.cs
@@ -15,7 +15,7 @@ public static Repository GetRepo(string gitDirectory)
             {
                 if (exception.Message.Contains("LibGit2Sharp.Core.NativeMethods") || exception.Message.Contains("FilePathMarshaler"))
                 {
-                    throw new ErrorException("Restart of the process may be required to load an updated version of LibGit2Sharp.");
+                    throw new WarningException("Restart of the process may be required to load an updated version of LibGit2Sharp.");
                 }
                 throw;
             }
diff --git a/GitVersionCore/SemanticVersion.cs b/GitVersionCore/SemanticVersion.cs
@@ -90,7 +90,7 @@ public static SemanticVersion Parse(string version)
         {
             SemanticVersion semanticVersion;
             if (!TryParse(version, out semanticVersion))
-                throw new ErrorException(string.Format("Failed to parse {0} into a Semantic Version", version));
+                throw new WarningException(string.Format("Failed to parse {0} into a Semantic Version", version));
 
             return semanticVersion;
         }
diff --git a/GitVersionCore/VersionCache.cs b/GitVersionCore/VersionCache.cs
@@ -14,7 +14,7 @@ public static SemanticVersion GetVersion(string gitDirectory)
                 var branch = repo.Head;
                 if (branch.Tip == null)
                 {
-                    throw new ErrorException("No Tip found. Has repo been initialized?");
+                    throw new WarningException("No Tip found. Has repo been initialized?");
                 }
 
                 var ticks = DirectoryDateFinder.GetLastDirectoryWrite(gitDirectory);
diff --git a/GitVersionCore/VersionOnMasterFinder.cs b/GitVersionCore/VersionOnMasterFinder.cs
@@ -94,7 +94,7 @@ static VersionPoint RetrieveStableVersionPointFor(IEnumerable<Tag> allTags, Comm
 
             if (tags.Count > 1)
             {
-                throw new ErrorException(
+                throw new WarningException(
                     string.Format("Commit '{0}' bears more than one stable tag: {1}",
                         c.Id.ToString(7), string.Join(", ", tags.Select(t => t.Name))));
             }
@@ -126,7 +126,7 @@ static Commit RetrieveMergeCommit(Tag stableTag)
             var target = stableTag.PeeledTarget();
             if (!(target is Commit))
             {
-                throw new ErrorException(
+                throw new WarningException(
                     string.Format("Target '{0}' of Tag '{1}' isn't a Commit.",
                         target.Id.ToString(7), stableTag.Name));
             }
diff --git a/GitVersionCore/WarningException.cs b/GitVersionCore/WarningException.cs
@@ -4,14 +4,14 @@ namespace GitVersion
     using System.Runtime.Serialization;
 
     [Serializable]
-    public class ErrorException : Exception
+    public class WarningException : Exception
     {
-        public ErrorException(string message)
+        public WarningException(string message)
             : base(message)
         {
         }
 
-        protected ErrorException(SerializationInfo info, StreamingContext context)
+        protected WarningException(SerializationInfo info, StreamingContext context)
             : base(info, context)
         {
         }
diff --git a/GitVersionExe/Program.cs b/GitVersionExe/Program.cs
@@ -81,7 +81,7 @@ static void Main()
                             string part;
                             if (!variables.TryGetValue(arguments.VersionPart, out part))
                             {
-                                throw new ErrorException(string.Format("Could not extract '{0}' from the available parts.", arguments.VersionPart));
+                                throw new WarningException(string.Format("Could not extract '{0}' from the available parts.", arguments.VersionPart));
                             }
                             Console.WriteLine(part);
                             break;
@@ -110,17 +110,17 @@ static void Main()
                     DeleteHelper.DeleteGitRepository(gitPreparer.DynamicGitRepositoryPath);
                 }
             }
-            catch (ErrorException exception)
+            catch (WarningException exception)
             {
                 var error = string.Format("An error occurred:\r\n{0}", exception.Message);
-                Logger.WriteError(error);
+                Logger.WriteWarning(error);
 
                 exitCode = 1;
             }
             catch (Exception exception)
             {
                 var error = string.Format("An unexpected error occurred:\r\n{0}", exception);
-                Logger.WriteWarning(error);
+                Logger.WriteError(error);
 
                 exitCode = 1;
             }
@@ -203,7 +203,7 @@ static bool RunMsBuildIfNeeded(Arguments args, string workingDirectory, Dictiona
                 GetEnvironmentalVariables(variables));
 
             if (results != 0)
-                throw new ErrorException("MsBuild execution failed, non-zero return code");
+                throw new WarningException("MsBuild execution failed, non-zero return code");
 
             return true;
         }
@@ -218,7 +218,7 @@ static bool RunExecCommandIfNeeded(Arguments args, string workingDirectory, Dict
                 null, args.Exec, args.ExecArgs, workingDirectory,
                 GetEnvironmentalVariables(variables));
             if (results != 0)
-                throw new ErrorException(string.Format("Execution of {0} failed, non-zero return code", args.Exec));
+                throw new WarningException(string.Format("Execution of {0} failed, non-zero return code", args.Exec));
 
             return true;
         }
diff --git a/GitVersionTask/GetVersion.cs b/GitVersionTask/GetVersion.cs
@@ -8,7 +8,6 @@
 
     public class GetVersion : Task
     {
-
         [Required]
         public string SolutionDirectory { get; set; }
 
@@ -100,10 +99,10 @@ public override bool Execute()
                 }
                 return true;
             }
-            catch (ErrorException errorException)
+            catch (WarningException errorException)
             {
-                logger.LogError(errorException.Message);
-                return false;
+                logger.LogWarning(errorException.Message);
+                return true;
             }
             catch (Exception exception)
             {
diff --git a/GitVersionTask/InvalidFileChecker.cs b/GitVersionTask/InvalidFileChecker.cs
@@ -12,7 +12,7 @@ public static void CheckForInvalidFiles(IEnumerable<ITaskItem> compileFiles, str
         {
             foreach (var compileFile in GetInvalidFiles(compileFiles, projectFile))
             {
-                throw new ErrorException("File contains assembly version attributes with conflict with the attributes generated by GitVersion" + compileFile);
+                throw new WarningException("File contains assembly version attributes with conflict with the attributes generated by GitVersion" + compileFile);
             }
         }
 
diff --git a/GitVersionTask/UpdateAssemblyInfo.cs b/GitVersionTask/UpdateAssemblyInfo.cs
diff --git a/GitVersionTask/WriteVersionInfoToBuildLog.cs b/GitVersionTask/WriteVersionInfoToBuildLog.cs
diff --git a/Tests/ArgumentParserTests.cs b/Tests/ArgumentParserTests.cs
diff --git a/Tests/BranchFinders/MasterTests.cs b/Tests/BranchFinders/MasterTests.cs
diff --git a/Tests/BranchFinders/PullBranchTests.cs b/Tests/BranchFinders/PullBranchTests.cs
diff --git a/Tests/BranchFinders/ReleaseTests.cs b/Tests/BranchFinders/ReleaseTests.cs
diff --git a/Tests/GitFlow/GitFlowVersionFinderTests.cs b/Tests/GitFlow/GitFlowVersionFinderTests.cs
diff --git a/Tests/UpdateAssemblyInfoTests.cs b/Tests/UpdateAssemblyInfoTests.cs
diff --git a/Tests/VersionOnMasterFinderTests.cs b/Tests/VersionOnMasterFinderTests.cs

Original file line number	Diff line number	Diff line change
`@@ -147,14 +147,14 @@ public static Arguments ParseArguments(List<string> commandLineArguments)`
`147`	`147`	`OutputType outputType;`
`148`	`148`	`if (!Enum.TryParse(value, true, out outputType))`
`149`	`149`	`{`
`150`		`- throw new ErrorException(string.Format("Value '{0}' cannot be parsed as output type, please use 'json' or 'buildserver'", value));`
	`150`	`+ throw new WarningException(string.Format("Value '{0}' cannot be parsed as output type, please use 'json' or 'buildserver'", value));`
`151`	`151`	`}`
`152`	`152`
`153`	`153`	`arguments.Output = outputType;`
`154`	`154`	`continue;`
`155`	`155`	`}`
`156`	`156`
`157`		`- throw new ErrorException(string.Format("Could not parse command line parameter '{0}'.", name));`
	`157`	`+ throw new WarningException(string.Format("Could not parse command line parameter '{0}'.", name));`
`158`	`158`	`}`
`159`	`159`	`return arguments;`
`160`	`160`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)`
`22`	`22`	`{`
`23`	`23`	`if (string.IsNullOrEmpty(gitDirectory))`
`24`	`24`	`{`
`25`		`- throw new ErrorException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");`
	`25`	`+ throw new WarningException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`var repoBranch = Environment.GetEnvironmentVariable("APPVEYOR_REPO_BRANCH");`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)`
`32`	`32`	`{`
`33`	`33`	`if (string.IsNullOrEmpty(gitDirectory))`
`34`	`34`	`{`
`35`		`- throw new ErrorException("Failed to find .git directory on agent");`
	`35`	`+ throw new WarningException("Failed to find .git directory on agent");`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`GitHelper.NormalizeGitDirectory(gitDirectory, arguments);`
Original file line number	Diff line number	Diff line change
`@@ -86,14 +86,14 @@ static void CreateFakeBranchPointingAtThePullRequestTip(Repository repo, Argumen`
`86`	`86`	`if (refs.Count == 0)`
`87`	`87`	`{`
`88`	`88`	`var message = string.Format("Couldn't find any remote tips from remote '{0}' pointing at the commit '{1}'.", remote.Url, headTipSha);`
`89`		`- throw new ErrorException(message);`
	`89`	`+ throw new WarningException(message);`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`if (refs.Count > 1)`
`93`	`93`	`{`
`94`	`94`	`var names = string.Join(", ", refs.Select(r => r.CanonicalName));`
`95`	`95`	`var message = string.Format("Found more than one remote tip from remote '{0}' pointing at the commit '{1}'. Unable to determine which one to use ({2}).", remote.Url, headTipSha, names);`
`96`		`- throw new ErrorException(message);`
	`96`	`+ throw new WarningException(message);`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`var canonicalName = refs[0].CanonicalName;`
`@@ -102,7 +102,7 @@ static void CreateFakeBranchPointingAtThePullRequestTip(Repository repo, Argumen`
`102`	`102`	`if (!canonicalName.StartsWith("refs/pull/"))`
`103`	`103`	`{`
`104`	`104`	`var message = string.Format("Remote tip '{0}' from remote '{1}' doesn't look like a valid pull request.", canonicalName, remote.Url);`
`105`		`- throw new ErrorException(message);`
	`105`	`+ throw new WarningException(message);`
`106`	`106`	`}`
`107`	`107`
`108`	`108`	`var fakeBranchName = canonicalName.Replace("refs/pull/", "refs/heads/pull/");`
`@@ -168,7 +168,7 @@ static Remote EnsureOnlyOneRemoteIsDefined(IRepository repo)`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`var message = string.Format("{0} remote(s) have been detected. When being run on a TeamCity agent, the Git repository is expected to bear one (and no more than one) remote.", howMany);`
`171`		`- throw new ErrorException(message);`
	`171`	`+ throw new WarningException(message);`
`172`	`172`	`}`
`173`	`173`	`}`
`174`	`174`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ public override void PerformPreProcessingSteps(string gitDirectory)`
`20`	`20`	`{`
`21`	`21`	`if (string.IsNullOrEmpty(gitDirectory))`
`22`	`22`	`{`
`23`		`- throw new ErrorException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");`
	`23`	`+ throw new WarningException("Failed to find .git directory on agent. Please make sure agent checkout mode is enabled for you VCS roots - http://confluence.jetbrains.com/display/TCD8/VCS+Checkout+Mode");`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`GitHelper.NormalizeGitDirectory(gitDirectory, arguments);`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ Commit FindCommonAncestorWithDevelop(IRepository repo, Branch branch, BranchType`
`65`	`65`	`return ancestor;`
`66`	`66`	`}`
`67`	`67`
`68`		`- throw new ErrorException(`
	`68`	`+ throw new WarningException(`
`69`	`69`	`string.Format("A {0} branch is expected to branch off of 'develop'. "`
`70`	`70`	`+ "However, branch 'develop' and '{1}' do not share a common ancestor."`
`71`	`71`	`, branchType, branch.Name));`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ public SemanticVersion FindVersion(IRepository repository, Commit tip)`
`26`	`26`	`}`
`27`	`27`	`}`
`28`	`28`
`29`		`- throw new ErrorException("The head of master should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");`
	`29`	`+ throw new WarningException("The head of master should always be a merge commit if you follow gitflow. Please create one or work around this by tagging the commit with SemVer compatible Id.");`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`SemanticVersion BuildVersion(IRepository repository, Commit tip, int major, int minor, int patch)`
Original file line number	Diff line number	Diff line change
`@@ -105,23 +105,23 @@ void EnsureVersionIsValid(SemanticVersion version, Branch branch, BranchType bra`
`105`	`105`
`106`	`106`	`if (version.PreReleaseTag.HasTag())`
`107`	`107`	`{`
`108`		`- throw new ErrorException(msg + string.Format("Supported format is '{0}-Major.Minor.Patch'.", branchType.ToString().ToLowerInvariant()));`
	`108`	`+ throw new WarningException(msg + string.Format("Supported format is '{0}-Major.Minor.Patch'.", branchType.ToString().ToLowerInvariant()));`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`switch (branchType)`
`112`	`112`	`{`
`113`	`113`	`case BranchType.Hotfix:`
`114`	`114`	`if (version.Patch == 0)`
`115`	`115`	`{`
`116`		`- throw new ErrorException(msg + "A patch segment different than zero is required.");`
	`116`	`+ throw new WarningException(msg + "A patch segment different than zero is required.");`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`break;`
`120`	`120`
`121`	`121`	`case BranchType.Release:`
`122`	`122`	`if (version.Patch != 0)`
`123`	`123`	`{`
`124`		`- throw new ErrorException(msg + "A patch segment equals to zero is required.");`
	`124`	`+ throw new WarningException(msg + "A patch segment equals to zero is required.");`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`break;`
`@@ -154,7 +154,7 @@ int NumberOfCommitsInBranchNotKnownFromBaseBranch(`
`154`	`154`	`if (ancestor == null)`
`155`	`155`	`{`
`156`	`156`	`var message = string.Format("A {0} branch is expected to branch off of '{1}'. However, branch '{1}' and '{2}' do not share a common ancestor.", branchType, baseBranchName, branch.Name);`
`157`		`- throw new ErrorException(message);`
	`157`	`+ throw new WarningException(message);`
`158`	`158`	`}`
`159`	`159`
`160`	`160`	`var filter = new CommitFilter`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ string ExtractIssueNumber(GitVersionContext context)`
`31`	`31`
`32`	`32`	`if (!LooksLikeAValidPullRequestNumber(issueNumber))`
`33`	`33`	`{`
`34`		`- throw new ErrorException(string.Format("Unable to extract pull request number from '{0}'.",`
	`34`	`+ throw new WarningException(string.Format("Unable to extract pull request number from '{0}'.",`
`35`	`35`	`pullRequestBranch.CanonicalName));`
`36`	`36`	`}`
`37`	`37`