From 90ff337f7cd3b00255d3c9f3f233d1c985811564 Mon Sep 17 00:00:00 2001 From: claude Date: Thu, 13 Aug 2026 03:07:34 -0700 Subject: [PATCH] fix: satisfy clippy -D warnings so CI can pass CI runs `cargo clippy -- -D warnings`, which already failed on main with four errors predating this branch: duplicated attribute src/models.rs, src/config.rs literal with an empty format src/cli_main.rs (x2) Both structs carried #[allow(dead_code)] twice; dropped the duplicate. The println! calls moved the literal into the format string as clippy suggested. No behaviour change -- output is byte-identical. Kept separate from the feature commit so it can be dropped or landed on its own. --- src/cli_main.rs | 4 ++-- src/config.rs | 1 - src/models.rs | 1 - 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/cli_main.rs b/src/cli_main.rs index 99f3a7b..9b65e30 100644 --- a/src/cli_main.rs +++ b/src/cli_main.rs @@ -65,7 +65,7 @@ fn main() { if users.is_empty() { println!("No users."); } else { - println!("{:<20} {}", "Username", "Created"); + println!("{:<20} Created", "Username"); println!("{}", "-".repeat(40)); for (u, c) in users { println!("{:<20} {}", u, c); @@ -115,7 +115,7 @@ fn main() { if keys.is_empty() { println!("No API keys."); } else { - println!("{:<20} {}", "Label", "Created"); + println!("{:<20} Created", "Label"); println!("{}", "-".repeat(40)); for (l, c) in keys { println!("{:<20} {}", l, c); diff --git a/src/config.rs b/src/config.rs index 72aef85..a31e702 100644 --- a/src/config.rs +++ b/src/config.rs @@ -1,7 +1,6 @@ use serde::Deserialize; use std::fs; -#[allow(dead_code)] #[derive(Debug, Deserialize, Clone)] #[allow(dead_code)] pub struct Config { diff --git a/src/models.rs b/src/models.rs index 9263e8b..6d3ba1c 100644 --- a/src/models.rs +++ b/src/models.rs @@ -27,7 +27,6 @@ pub struct SubmitAction { pub run_as: Option, } -#[allow(dead_code)] #[derive(Debug, Serialize)] #[allow(dead_code)] pub struct ActionResult {