Merge exec_result display and status into single match
Assisted-by: GLM-5.1 via pi
change oynlxmvmylomkxptuxsxvkmowpvmtkpn
commit 3cd0265a284d6acffefa1782cccfa1ac1e872886
author Alpha Chen <alpha@kejadlen.dev>
date
parent twmkvztk
diff --git a/src/bin/quire/commands/ci.rs b/src/bin/quire/commands/ci.rs
index f5c0264..249dfcb 100644
--- a/src/bin/quire/commands/ci.rs
+++ b/src/bin/quire/commands/ci.rs
@@ -78,29 +78,25 @@ pub async fn run(quire: &Quire, maybe_sha: Option<&str>) -> Result<()> {
 
     let exec_result = run.execute(pipeline, secrets);
 
-    // Display outputs from completed jobs.
-    if let Ok(ref outputs) = exec_result {
-        for (job_id, job_outputs) in outputs {
-            if job_outputs.is_empty() {
-                continue;
-            }
-            println!("\n==> {}", job_id);
-            for o in job_outputs {
-                if !o.stdout.is_empty() {
-                    print!("{}", o.stdout);
-                }
-                if !o.stderr.is_empty() {
-                    eprint!("{}", o.stderr);
+    match exec_result {
+        Ok(outputs) => {
+            for (job_id, job_outputs) in &outputs {
+                if job_outputs.is_empty() {
+                    continue;
                 }
-                if o.exit != 0 {
-                    println!("(exit {})", o.exit);
+                println!("\n==> {}", job_id);
+                for o in job_outputs {
+                    if !o.stdout.is_empty() {
+                        print!("{}", o.stdout);
+                    }
+                    if !o.stderr.is_empty() {
+                        eprint!("{}", o.stderr);
+                    }
+                    if o.exit != 0 {
+                        println!("(exit {})", o.exit);
+                    }
                 }
             }
-        }
-    }
-
-    match exec_result {
-        Ok(_) => {
             println!("\nRun complete.");
             Ok(())
         }