Unnamed repository; edit this file 'description' to name the repository.
Merge pull request #20096 from lnicola/silence-rustfmt-failed
internal: Don't show notifications on failed rustfmt calls
Lukas Wirth 10 months ago
parent 01339d4 · parent 4c07f8b · commit 749288e
-rw-r--r--crates/rust-analyzer/src/handlers/request.rs19
1 files changed, 8 insertions, 11 deletions
diff --git a/crates/rust-analyzer/src/handlers/request.rs b/crates/rust-analyzer/src/handlers/request.rs
index afd9eff526..a76a65220d 100644
--- 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)
}
};
}