From d03f8d8c52f608c205ed064e33f94123b14b4979 Mon Sep 17 00:00:00 2001 From: Perry Hertler Date: Thu, 20 Aug 2026 10:52:44 -0500 Subject: [PATCH 1/2] chore: bump Rust toolchain to 1.97.1 Moves the pinned channel from 1.89.0 (Aug 2025) to 1.97.1. Co-Authored-By: Claude Fable 5 --- rust-toolchain.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust-toolchain.toml b/rust-toolchain.toml index ff1a27d..efba89c 100644 --- a/rust-toolchain.toml +++ b/rust-toolchain.toml @@ -1,4 +1,4 @@ [toolchain] -channel = "1.89.0" +channel = "1.97.1" components = ["clippy", "rustfmt"] targets = ["x86_64-apple-darwin", "aarch64-apple-darwin", "x86_64-unknown-linux-gnu"] From a203f3959f6b1ce357f3886d926afa6a06f3b7f7 Mon Sep 17 00:00:00 2001 From: Perry Hertler Date: Thu, 20 Aug 2026 10:53:07 -0500 Subject: [PATCH 2/2] style: drop redundant refs in format! arguments Rust 1.97's clippy extends useless_borrows_in_formatting to catch `&` on Display-able expressions in format args. CI builds with RUSTFLAGS=-Dwarnings, so these 8 sites would fail the clippy job. Applied via `cargo clippy --fix`; removing the borrows only, no behavior change. Co-Authored-By: Claude Fable 5 --- src/cli.rs | 2 +- src/ownership/validator.rs | 2 +- src/runner.rs | 10 +++++----- tests/common/mod.rs | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/cli.rs b/src/cli.rs index e3e3c08..1d0a7f9 100644 --- a/src/cli.rs +++ b/src/cli.rs @@ -85,7 +85,7 @@ impl Args { fn absolute_project_root(&self) -> Result> { self.project_root.canonicalize().change_context(RunnerError::Io(format!( "Can't canonicalize project root: {}", - &self.project_root.to_string_lossy() + self.project_root.to_string_lossy() ))) } diff --git a/src/ownership/validator.rs b/src/ownership/validator.rs index a3896cd..65fb648 100644 --- a/src/ownership/validator.rs +++ b/src/ownership/validator.rs @@ -247,7 +247,7 @@ impl Display for Errors { let messages = errors.iter().flat_map(|error| error.messages()).sorted().join("\n"); if !messages.is_empty() { writeln!(f)?; - write!(f, "{}", &messages)?; + write!(f, "{}", messages)?; } writeln!(f)?; diff --git a/src/runner.rs b/src/runner.rs index f84e034..c69f712 100644 --- a/src/runner.rs +++ b/src/runner.rs @@ -89,22 +89,22 @@ impl Runner { GlobalCache::new(run_config.project_root.clone(), config.cache_directory.clone()) .change_context(Error::Io(format!( "Can't create cache: {}", - &run_config.config_path.to_string_lossy() + run_config.config_path.to_string_lossy() ))) - .attach(format!("Can't create cache: {}", &run_config.config_path.to_string_lossy()))? + .attach(format!("Can't create cache: {}", run_config.config_path.to_string_lossy()))? .into() }; let mut project_builder = ProjectBuilder::new(&config, run_config.project_root.clone(), codeowners_file_path.clone(), &cache); let project = project_builder.build().change_context(Error::Io(format!( "Can't build project: {}", - &run_config.config_path.to_string_lossy() + run_config.config_path.to_string_lossy() )))?; let ownership = Ownership::build(project); cache.persist_cache().change_context(Error::Io(format!( "Can't persist cache: {}", - &run_config.config_path.to_string_lossy() + run_config.config_path.to_string_lossy() )))?; Ok(Self { @@ -250,7 +250,7 @@ impl Runner { pub fn delete_cache(&self) -> RunResult { match self.cache.delete_cache().change_context(Error::Io(format!( "Can't delete cache: {}", - &self.run_config.config_path.to_string_lossy() + self.run_config.config_path.to_string_lossy() ))) { Ok(_) => RunResult::default(), Err(err) => RunResult { diff --git a/tests/common/mod.rs b/tests/common/mod.rs index 0500841..8c55df0 100644 --- a/tests/common/mod.rs +++ b/tests/common/mod.rs @@ -49,7 +49,7 @@ pub fn teardown() { .filter_map(Result::ok) .for_each(|cache_dir| { if let Err(err) = fs::remove_dir_all(&cache_dir) { - eprintln!("Failed to remove {} during test teardown: {}", &cache_dir.display(), err); + eprintln!("Failed to remove {} during test teardown: {}", cache_dir.display(), err); } }); }