Stop swallowing the exit status in InitArgs::run_check

Summary: The only caller of run_check returns success regardless of the run_check status, so there's no need for run_check to discard its status: https://github.com/facebook/pyrefly/blob/785a8effd660a36e08078df8e426a71e1feafb60/pyrefly/lib/commands/init.rs#L78.

Reviewed By: kinto0

Differential Revision: D89905963

fbshipit-source-id: 0fa5b726b0c63050ff3fc2c210bbfe98ce1f09f9

Author: rchen152

pyrefly/lib/commands/init.rs

@@ -107,13 +107,11 @@ impl InitArgs {
             FilesArgs::get(Vec::new(), config_path, ConfigOverrideArgs::default())?;
 
         // Run the check directly
-        match check_args.run_once(filtered_globs, config_finder) {
-            Ok((status, errors)) => Ok((status, errors)),
-            Err(e) => {
-                error!("Failed to run pyrefly check: {}", e);
-                Ok((CommandExitStatus::Success, Vec::new())) // Still return success to match original behavior
-            }
+        let res = check_args.run_once(filtered_globs, config_finder);
+        if let Err(e) = &res {
+            error!("Failed to run pyrefly check: {}", e);
         }
+        res
     }
 
     fn prompt_error_suppression(