From 7053561b633c5b703e618322de97a35f87f390d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fernando=20P=C3=A9rez?= Date: Fri, 30 Jan 2026 10:01:54 +0100 Subject: [PATCH] docs: Document default for include_warnings (#47838) ## Summary Add documentation for the default value of `include_warnings` in diagnostics settings. This makes it consistent with other settings that document their default values. ## Test plan - Documentation-only change, no functional changes Release Notes: - N/A --- assets/settings/default.json | 2 ++ crates/settings_content/src/project.rs | 2 ++ 2 files changed, 4 insertions(+) diff --git a/assets/settings/default.json b/assets/settings/default.json index 6f0c505fb4113e527797bb6ea9a153575f4b7bdf..bd41f1704f6be57921202a71fe229729317e204f 100644 --- a/assets/settings/default.json +++ b/assets/settings/default.json @@ -1376,6 +1376,8 @@ // Whether to show the project diagnostics button in the status bar. "button": true, // Whether to show warnings or not by default. + // + // Default: true "include_warnings": true, // Settings for using LSP pull diagnostics mechanism in Zed. "lsp_pull_diagnostics": { diff --git a/crates/settings_content/src/project.rs b/crates/settings_content/src/project.rs index 0273724c9e19cd1a88081b5deec7438a8e08041a..be6ad65ac4690375829a20dd08e5b28833672968 100644 --- a/crates/settings_content/src/project.rs +++ b/crates/settings_content/src/project.rs @@ -526,6 +526,8 @@ pub struct DiagnosticsSettingsContent { pub button: Option, /// Whether or not to include warning diagnostics. + /// + /// Default: true pub include_warnings: Option, /// Settings for using LSP pull diagnostics mechanism in Zed.