refactor(link): add Ok and make IoError more understandable

Signed-off-by: Christina Sørensen <christina@cafkafk.com>
cafkafk · Oct 17, 2023 · 4a0f855 · 4a0f855
1 parent f17e2a5
commit 4a0f855
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/git.rs b/src/git.rs
@@ -598,6 +598,10 @@ impl Config {
                             let mut sp =
                                 Spinner::new(Spinners::Dots10, format!("{}: {}", link.name, op));
                             match f(link) {
+                                Ok(_) => sp.stop_and_persist(
+                                    success_str(),
+                                    format!("{}: {}", link.name, op),
+                                ),
                                 Err(e @ LinkError::AlreadyLinked(_, _)) => {
                                     sp.stop_and_persist(success_str(), format!("{e}"))
                                 }
@@ -613,9 +617,10 @@ impl Config {
                                 Err(e @ LinkError::FailedCreatingLink(_, _)) => {
                                     sp.stop_and_persist(failure_str(), format!("{e}"))
                                 }
-                                Err(e @ LinkError::IoError(_)) => {
-                                    sp.stop_and_persist(failure_str(), format!("{e}"))
-                                }
+                                Err(e @ LinkError::IoError(_)) => sp.stop_and_persist(
+                                    failure_str(),
+                                    format!("{}: {op}, {e}", link.name),
+                                ),
                                 Err(e) => sp.stop_and_persist(failure_str(), format!("{e}")),
                                 _ => sp.stop_and_persist(
                                     failure_str(),