Moved nofetch option so it doesn't skip normalisation entirely when specified

Author: JakeGinnivan

src/GitVersionCore/BuildServers/GitHelper.cs

@@ -12,13 +12,6 @@ public static class GitHelper
 
         public static void NormalizeGitDirectory(string gitDirectory, Authentication authentication, bool noFetch)
         {
-            //If noFetch is enabled, then GitVersion will assume that the git repository is normalized before execution, so that fetching from remotes is not required.
-            if (noFetch)
-            {
-                Logger.WriteInfo("Skipping fetching");
-                return;
-            }
-
             using (var repo = new Repository(gitDirectory))
             {
                 var remote = EnsureOnlyOneRemoteIsDefined(repo);
@@ -28,8 +21,14 @@ public static void NormalizeGitDirectory(string gitDirectory, Authentication aut
                 Logger.WriteInfo(string.Format("Fetching from remote '{0}' using the following refspecs: {1}.",
                     remote.Name, string.Join(", ", remote.FetchRefSpecs.Select(r => r.Specification))));
 
-                var fetchOptions = BuildFetchOptions(authentication.Username, authentication.Password);
-                repo.Network.Fetch(remote, fetchOptions);
+    
+                //If noFetch is enabled, then GitVersion will assume that the git repository is normalized before execution, so that fetching from remotes is not required.
+                if (!noFetch)
+                {
+                    Logger.WriteInfo("Skipping fetching");
+                    var fetchOptions = BuildFetchOptions(authentication.Username, authentication.Password);
+                    repo.Network.Fetch(remote, fetchOptions);
+                }
 
                 CreateMissingLocalBranchesFromRemoteTrackingOnes(repo, remote.Name);