Merge fixes/git-save-path-failure-cleanup into fixes/mac-path-variable

Author: shana

src/GitHub.Api/Git/GitClient.cs

@@ -112,11 +112,6 @@ public GitClient(IEnvironment environment, IProcessManager processManager, Cance
 
         public ITask<ValidateGitInstallResult> ValidateGitInstall(NPath path, bool isCustomGit)
         {
-            if (!path.FileExists())
-            {
-                return new FuncTask<ValidateGitInstallResult>(TaskEx.FromResult(new ValidateGitInstallResult(false, null, null)));
-            }
-
             Version gitVersion = null;
             Version gitLfsVersion = null;
 
@@ -125,25 +120,20 @@ public ITask<ValidateGitInstallResult> ValidateGitInstall(NPath path, bool isCus
                     gitVersion?.CompareTo(Constants.MinimumGitVersion) >= 0 &&
                     gitLfsVersion?.CompareTo(Constants.MinimumGitLfsVersion) >= 0,
                     gitVersion, gitLfsVersion));
-              
-            var gitLfsVersionTask = new GitLfsVersionTask(cancellationToken)
-                .Configure(processManager, path, dontSetupGit: isCustomGit);
-            
-            gitLfsVersionTask
-                .Then((result, version) => {return gitLfsVersion = version;})
-                .Then(endTask, taskIsTopOfChain: true);
-
-            gitLfsVersionTask.Then(endTask, TaskRunOptions.OnFailure, taskIsTopOfChain:true);
-
-            var gitVersionTask = new GitVersionTask(cancellationToken)
-                .Configure(processManager, path, dontSetupGit: isCustomGit);
-
-            gitVersionTask
-                .Then((result, version) => { return gitVersion = version; })
-                .Then(gitLfsVersionTask, taskIsTopOfChain: true);
-            
-            gitVersionTask.Then(endTask, TaskRunOptions.OnFailure, taskIsTopOfChain:true);
 
+            if (path.FileExists())
+            {
+                var gitLfsVersionTask = new GitLfsVersionTask(cancellationToken)
+                    .Configure(processManager, path, dontSetupGit: isCustomGit);
+                gitLfsVersionTask.OnEnd += (t, v, _, __) => gitLfsVersion = v;
+                var gitVersionTask = new GitVersionTask(cancellationToken)
+                    .Configure(processManager, path, dontSetupGit: isCustomGit);
+                gitVersionTask.OnEnd += (t, v, _, __) => gitVersion = v;
+
+                gitVersionTask
+                    .Then(gitLfsVersionTask)
+                    .Finally(endTask);
+            }
             return endTask;
         }
 

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/GitPathView.cs

@@ -270,7 +270,7 @@ private void ValidateAndSetGitInstallPath(string value)
                 gitVersionErrorMessage = null;
 
                 GitClient.ValidateGitInstall(value.ToNPath(), true)
-                    .FinallyInUI((success, exception, result) =>
+                    .ThenInUI((success, result) =>
                     {
                         if (!success)
                         {