chore: cleaner way to check status code

cli/src/build.rs

@@ -214,14 +214,15 @@ pub fn build_file<'a>(
 
     let total_time = compilation_start.elapsed().unwrap();
 
-    // TODO change this to report whether there were errors or warnings!
-    let outcome = match &cmd_result {
-        Ok(exit_status) if exit_status.success() => BuildOutcome::NoProblems,
-        _ => BuildOutcome::Errors,
-    };
-
     // If the cmd errored out, return the Err.
-    cmd_result?;
+    let exit_status = cmd_result?;
+
+    // TODO change this to report whether there were errors or warnings!
+    let outcome = if exit_status.success() {
+        BuildOutcome::NoProblems
+    } else {
+        BuildOutcome::Errors
+    };
 
     Ok(BuiltFile {
         binary_path,