Merge pull request #19636 from Veykril/push-wttkunmqyskm

fix: Fix completion_snippets_custom config always erroring
This commit is contained in:
Lukas Wirth 2025-04-21 06:28:45 +00:00 committed by GitHub
commit 66e3b5819e
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -923,7 +923,6 @@ impl Config {
if !(json.is_null() || json.as_object().is_some_and(|it| it.is_empty())) {
let detached_files = get_field_json::<Vec<Utf8PathBuf>>(
&mut json,
// Do not record errors here; it is not an error if a field is missing here.
&mut Vec::new(),
"detachedFiles",
None,
@ -935,19 +934,20 @@ impl Config {
patch_old_style::patch_json_for_outdated_configs(&mut json);
let snips = get_field_json::<FxIndexMap<String, SnippetDef>>(
&mut json,
// Do not record errors here; it is not an error if a field is missing here.
&mut Vec::new(),
"completion_snippets_custom",
None,
)
.unwrap_or(self.completion_snippets_custom().to_owned());
let mut json_errors = vec![];
let input = FullConfigInput::from_json(json, &mut json_errors);
// IMPORTANT : This holds as long as ` completion_snippets_custom` is declared `client`.
config.snippets.clear();
let snips = input
.global
.completion_snippets_custom
.as_ref()
.unwrap_or(&self.default_config.global.completion_snippets_custom);
#[allow(dead_code)]
let _ = Self::completion_snippets_custom;
for (name, def) in snips.iter() {
if def.prefix.is_empty() && def.postfix.is_empty() {
continue;
@ -974,8 +974,9 @@ impl Config {
)),
}
}
config.client_config = (
FullConfigInput::from_json(json, &mut json_errors),
input,
ConfigErrors(
json_errors
.into_iter()