From 3bb6d1456d64acc1b43278bc255f362f97b15714 Mon Sep 17 00:00:00 2001 From: oxalica Date: Tue, 10 Sep 2024 01:22:46 -0400 Subject: [PATCH] Enforce lint reasons --- Cargo.toml | 1 + blahctl/src/main.rs | 3 +-- blahd/tests/basic.rs | 6 ++---- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index 5da8b87..4a8cf36 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -8,6 +8,7 @@ members = [ default-members = ["blahd"] [workspace.lints.clippy] +allow_attributes = "warn" dbg_macro = "warn" print_stderr = "warn" print_stdout = "warn" diff --git a/blahctl/src/main.rs b/blahctl/src/main.rs index 5660afb..78330c4 100644 --- a/blahctl/src/main.rs +++ b/blahctl/src/main.rs @@ -56,14 +56,13 @@ enum Command { } #[derive(Debug, clap::Subcommand)] -#[allow(clippy::large_enum_variant)] enum DbCommand { /// Create and initialize database. Init, /// Set user property, possibly adding new users. SetUser { #[command(flatten)] - user: User, + user: Box, #[arg(long, value_parser = flag_parser::)] permission: ServerPermission, diff --git a/blahd/tests/basic.rs b/blahd/tests/basic.rs index b398fd8..930cd98 100644 --- a/blahd/tests/basic.rs +++ b/blahd/tests/basic.rs @@ -1,7 +1,5 @@ -// FIXME: False positive? -#![allow(clippy::unwrap_used)] -// Easy to use for fixtures. -#![allow(clippy::toplevel_ref_arg)] +#![expect(clippy::unwrap_used, reason = "FIXME: random false positive")] +#![expect(clippy::toplevel_ref_arg, reason = "easy to use for fixtures")] use std::fmt; use std::future::IntoFuture; use std::sync::{Arc, LazyLock};