show skipped repositories

Author: JC-Chung

src/ViewModels/Launcher.cs

@@ -133,6 +133,7 @@ public async Task FetchAllRepositoriesAsync()
             var total = 0;
             var succeeded = 0;
             var noRemoteNames = new List<string>();
+            var skippedNames = new List<string>();
             var failedNames = new List<string>();
 
             foreach (var page in pages)
@@ -147,17 +148,31 @@ public async Task FetchAllRepositoriesAsync()
                 }
 
                 total++;
-                if (await repo.FetchAllRemotesAsync())
-                    succeeded++;
-                else
-                    failedNames.Add(page.Node.Name);
+                var result = await repo.FetchAllRemotesAsync();
+                switch (result.Status)
+                {
+                    case Repository.FetchAllStatus.Succeeded:
+                        succeeded++;
+                        break;
+                    case Repository.FetchAllStatus.Skipped:
+                        skippedNames.Add(page.Node.Name);
+                        total--;
+                        break;
+                    case Repository.FetchAllStatus.Failed:
+                        var remotesInfo = string.Join(", ", result.FailedRemotes);
+                        failedNames.Add($"{page.Node.Name} (remote: {remotesInfo})");
+                        break;
+                }
             }
 
             var message = $"Fetched {succeeded}/{total} repositories";
 
             if (noRemoteNames.Count > 0)
                 message += $"\n{noRemoteNames.Count} repositories skipped (no remote):\n    {string.Join("\n    ", noRemoteNames)}";
 
+            if (skippedNames.Count > 0)
+                message += $"\n{skippedNames.Count} repositories skipped:\n    {string.Join("\n    ", skippedNames)}";
+
             if (failedNames.Count > 0)
                 message += $"\n{failedNames.Count} repositories failed:\n    {string.Join("\n    ", failedNames)}";
 

src/ViewModels/Repository.cs

@@ -650,44 +650,56 @@ public async Task FetchAsync(bool autoStart)
                 ShowPopup(new Fetch(this));
         }
 
-        public async Task<bool> FetchAllRemotesAsync()
+        public enum FetchAllStatus
         {
+            Succeeded,
+            Failed,
+            Skipped,
+        }
+
+        public class FetchAllRemotesResult
+        {
+            public FetchAllStatus Status { get; set; } = FetchAllStatus.Succeeded;
+            public List<string> FailedRemotes { get; } = new List<string>();
+        }
+
+        public async Task<FetchAllRemotesResult> FetchAllRemotesAsync()
+        {
+            var result = new FetchAllRemotesResult();
+
             if (IsAutoFetching)
-                return false;
+            {
+                result.Status = FetchAllStatus.Skipped;
+                return result;
+            }
 
             CommandLog log = null;
-            var succeeded = true;
 
             try
             {
-                var lockFile = Path.Combine(GitDir, "index.lock");
-                if (File.Exists(lockFile))
-                    return false;
-
-                if (_remotes.Count == 0)
-                    return false;
-
                 IsAutoFetching = true;
                 log = CreateLog("Fetch");
 
                 foreach (var remote in _remotes)
                 {
                     var succ = await new Commands.Fetch(FullPath, remote).Use(log).ExecAsync();
                     if (!succ)
-                        succeeded = false;
+                        result.FailedRemotes.Add(remote.Name);
                 }
 
                 _lastFetchTime = DateTime.Now;
             }
             catch
             {
                 // Ignore all exceptions.
-                succeeded = false;
+                result.FailedRemotes.Add("*");
             }
 
             IsAutoFetching = false;
             log?.Complete();
-            return succeeded;
+
+            result.Status = result.FailedRemotes.Count == 0 ? FetchAllStatus.Succeeded : FetchAllStatus.Failed;
+            return result;
         }
 
         public async Task PullAsync(bool autoStart)