Merge pull request #20096 from lnicola/silence-rustfmt-failed

internal: Don't show notifications on failed rustfmt calls
2025-09-28 11:20:54 +00:00 · 2025-06-25 13:03:21 +00:00 · 2025-06-25 13:03:21 +00:00 · 749288e0de
commit 749288e0de
parent 01339d4deb 4c07f8b25a
1 changed files with 8 additions and 11 deletions
--- a/crates/rust-analyzer/src/handlers/request.rs
+++ b/crates/rust-analyzer/src/handlers/request.rs
@ -2433,17 +2433,14 @@ fn run_rustfmt(
            }
            _ => {
                // Something else happened - e.g. `rustfmt` is missing or caught a signal
-                Err(LspError::new(
-                    -32900,
-                    format!(
-                        r#"rustfmt exited with:
-                           Status: {}
-                           stdout: {captured_stdout}
-                           stderr: {captured_stderr}"#,
-                        output.status,
-                    ),
-                )
-                .into())
+                tracing::error!(
+                    ?command,
+                    %output.status,
+                    %captured_stdout,
+                    %captured_stderr,
+                    "rustfmt failed"
+                );
+                Ok(None)
            }
        };
    }