Merge pull request #50 from epage/template

chore: Update from _rust template

.github/workflows/audit.yml

@@ -47,7 +47,7 @@ jobs:
           - bans licenses sources
     steps:
     - uses: actions/checkout@v4
-    - uses: EmbarkStudios/cargo-deny-action@v1
+    - uses: EmbarkStudios/cargo-deny-action@v2
       with:
         command: check ${{ matrix.checks }}
         rust-version: stable

.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/[email protected]

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"

src/lib.rs

@@ -241,7 +241,7 @@ impl Line {
     ) -> Result<(), std::io::Error> {
         match self {
             Self::Control { name, args } => {
-                write!(out, ".{}", name)?;
+                write!(out, ".{name}")?;
                 for arg in args {
                     write!(out, " {}", &escape_spaces(arg))?;
                 }
@@ -267,9 +267,9 @@ impl Line {
                             };
                             let text = escape_leading_cc(&text);
                             if let Inline::Bold(_) = inline {
-                                write!(out, r"\fB{}\fR", text)?;
+                                write!(out, r"\fB{text}\fR")?;
                             } else if let Inline::Italic(_) = inline {
-                                write!(out, r"\fI{}\fR", text)?;
+                                write!(out, r"\fI{text}\fR")?;
                             } else {
                                 if at_line_start && starts_with_cc(&text) {
                                     // Line would start with a period, so we
@@ -285,7 +285,7 @@ impl Line {
                                     // line.
                                     write!(out, r"\&").unwrap();
                                 }
-                                write!(out, "{}", text)?;
+                                write!(out, "{text}")?;
                             }
                         }
                     }
@@ -307,7 +307,7 @@ fn starts_with_cc(line: &str) -> bool {
 /// quotes in any way: there doesn't seem to a way to escape them.
 fn escape_spaces(w: &str) -> String {
     if w.contains(' ') {
-        format!("\"{}\"", w)
+        format!("\"{w}\"")
     } else {
         w.to_owned()
     }

tests/testsuite/demo.rs

@@ -78,8 +78,7 @@ pub(crate) fn has_command(command: &str) -> bool {
             // CI is expected to support all of the commands
             if is_ci() && cfg!(target_os = "linux") {
                 panic!(
-                    "expected command `{}` to be somewhere in PATH: {}",
-                    command, e
+                    "expected command `{command}` to be somewhere in PATH: {e}"
                 );
             }
             return false;
@@ -99,8 +98,7 @@ pub(crate) fn has_command(command: &str) -> bool {
     let stdout = String::from_utf8_lossy(&output.stdout);
     println!(
         "$ {command} --version
-{}",
-        stdout
+{stdout}"
     );
     if cfg!(target_os = "macos") && stdout.starts_with("GNU bash, version 3") {
         return false;