Skip to content

Commit

Permalink
Touch-up error messages in server file discovery (#11285)
Browse files Browse the repository at this point in the history
## Summary

Just making these messages a little more consistent with how we format
them in Ruff, uv, etc.
  • Loading branch information
charliermarsh authored May 5, 2024
1 parent a8a9729 commit 9db11dc
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 15 deletions.
4 changes: 1 addition & 3 deletions crates/ruff_server/src/session/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -274,9 +274,7 @@ impl ResolvedClientSettings {
.map(|rule| RuleSelector::from_str(rule).ok())
.collect()
}),
exclude: Self::resolve_optional(all_settings, |settings| {
Some(settings.exclude.as_ref()?.clone())
}),
exclude: Self::resolve_optional(all_settings, |settings| settings.exclude.clone()),
line_length: Self::resolve_optional(all_settings, |settings| settings.line_length),
configuration_preference: Self::resolve_or(
all_settings,
Expand Down
24 changes: 12 additions & 12 deletions crates/ruff_server/src/session/workspace/ruff_settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,9 @@ use crate::session::settings::{ConfigurationPreference, ResolvedEditorSettings};

#[derive(Default)]
pub(crate) struct RuffSettings {
// settings to pass into the ruff linter
/// Settings to pass into the Ruff linter.
linter: ruff_linter::settings::LinterSettings,
// settings to pass into the ruff formatter
/// Settings to pass into the Ruff formatter.
formatter: ruff_workspace::FormatterSettings,
}

Expand Down Expand Up @@ -118,7 +118,10 @@ impl RuffSettingsIndex {
return settings.clone();
}

tracing::info!("No ruff settings file (pyproject.toml/ruff.toml/.ruff.toml) found for {} - falling back to default configuration", document_path.display());
tracing::info!(
"No Ruff settings file found for {}; falling back to default configuration",
document_path.display()
);

self.fallback.clone()
}
Expand All @@ -127,10 +130,7 @@ impl RuffSettingsIndex {
struct EditorConfigurationTransformer<'a>(&'a ResolvedEditorSettings, &'a Path);

impl<'a> ConfigurationTransformer for EditorConfigurationTransformer<'a> {
fn transform(
&self,
filesystem_configuration: ruff_workspace::configuration::Configuration,
) -> ruff_workspace::configuration::Configuration {
fn transform(&self, filesystem_configuration: Configuration) -> Configuration {
let ResolvedEditorSettings {
configuration,
format_preview,
Expand All @@ -152,13 +152,13 @@ impl<'a> ConfigurationTransformer for EditorConfigurationTransformer<'a> {
select,
extend_select: extend_select.unwrap_or_default(),
ignore: ignore.unwrap_or_default(),
..Default::default()
..RuleSelection::default()
}],
..Default::default()
..LintConfiguration::default()
},
format: FormatConfiguration {
preview: format_preview.map(PreviewMode::from),
..Default::default()
..FormatConfiguration::default()
},
exclude: exclude.map(|exclude| {
exclude
Expand All @@ -170,15 +170,15 @@ impl<'a> ConfigurationTransformer for EditorConfigurationTransformer<'a> {
.collect()
}),
line_length,
..Default::default()
..Configuration::default()
};

// Merge in the editor-specified configuration file, if it exists.
let editor_configuration = if let Some(config_file_path) = configuration {
match open_configuration_file(&config_file_path, project_root) {
Ok(config_from_file) => editor_configuration.combine(config_from_file),
Err(err) => {
tracing::error!("Unable to find editor-specified configuration file {err}");
tracing::error!("Unable to find editor-specified configuration file: {err}");
editor_configuration
}
}
Expand Down

0 comments on commit 9db11dc

Please sign in to comment.