Merge pull request #175 from epage/template

chore: Update from _rust template

Author: epage

.github/workflows/pre-commit.yml

@@ -24,4 +24,6 @@ jobs:
     steps:
     - uses: actions/checkout@v4
     - uses: actions/setup-python@v5
+      with:
+        python-version: '3.x'
     - uses: pre-commit/action@v3.0.1

CONTRIBUTING.md

@@ -45,7 +45,7 @@ We ask that commits are atomic, meaning they are complete and have a single resp
 PRs should tell a cohesive story, with test and refactor commits that keep the
 fix or feature commits simple and clear.
 
-Specifically, we would encouage
+Specifically, we would encourage
 - File renames be isolated into their own commit
 - Add tests in a commit before their feature or fix, showing the current behavior.
   The diff for the feature/fix commit will then show how the behavior changed,

Cargo.toml

@@ -80,6 +80,7 @@ string_lit_as_bytes = "warn"
 string_to_string = "warn"
 todo = "warn"
 trait_duplication_in_bounds = "warn"
+uninlined_format_args = "warn"
 verbose_file_reads = "warn"
 wildcard_imports = "warn"
 zero_sized_map_values = "warn"

examples/case_tree.rs

@@ -7,7 +7,7 @@ fn main() {
     let var = 5;
     let case = pred.find_case(true, &var);
     if let Some(case) = case {
-        println!("var is {}", var);
+        println!("var is {var}");
         println!("{}", case.tree());
     }
 }

src/ord.rs

@@ -26,7 +26,7 @@ impl fmt::Display for EqOps {
             EqOps::Equal => "==",
             EqOps::NotEqual => "!=",
         };
-        write!(f, "{}", op)
+        write!(f, "{op}")
     }
 }
 
@@ -148,7 +148,7 @@ impl fmt::Display for OrdOps {
             OrdOps::GreaterThanOrEqual => ">=",
             OrdOps::GreaterThan => ">",
         };
-        write!(f, "{}", op)
+        write!(f, "{op}")
     }
 }
 

src/path/ft.rs

@@ -54,7 +54,7 @@ impl fmt::Display for FileType {
             FileType::Dir => "dir",
             FileType::Symlink => "symlink",
         };
-        write!(f, "{}", t)
+        write!(f, "{t}")
     }
 }
 

src/str/difference.rs

@@ -31,8 +31,8 @@ impl Predicate<str> for DifferencePredicate {
             None
         } else {
             let palette = crate::Palette::new(true);
-            let orig: Vec<_> = self.orig.lines().map(|l| format!("{}\n", l)).collect();
-            let variable: Vec<_> = variable.lines().map(|l| format!("{}\n", l)).collect();
+            let orig: Vec<_> = self.orig.lines().map(|l| format!("{l}\n")).collect();
+            let variable: Vec<_> = variable.lines().map(|l| format!("{l}\n")).collect();
             let diff = difflib::unified_diff(
                 &orig,
                 &variable,