Skip to content

Commit 55bc319

Browse files
committed
chore(dylint): bump perfectionist to d793ada and drop lint_silence_reason suppressions
perfectionist PR 172 fixes the lint_silence_reason false positive on `clap`-derive-generated `#[allow]` attributes (#430). Point the dylint library at the fix commit and remove the now-unnecessary module-level suppressions in src/args.rs and cli/ai_instructions.rs. Dylint passes with the suppressions removed. https://claude.ai/code/session_015ZyokLaZvcy5Yt8KpPxV3F
1 parent a04c97a commit 55bc319

3 files changed

Lines changed: 1 addition & 16 deletions

File tree

cli/ai_instructions.rs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,5 @@
11
#![cfg_attr(dylint_lib = "perfectionist", feature(register_tool))]
22
#![cfg_attr(dylint_lib = "perfectionist", register_tool(perfectionist))]
3-
#![cfg_attr(
4-
dylint_lib = "perfectionist",
5-
allow(
6-
perfectionist::lint_silence_reason,
7-
reason = "false positive on `clap`-derive-generated `#[allow]` attributes; see https://github.com/KSXGitHub/parallel-disk-usage/issues/430"
8-
)
9-
)]
103

114
use clap::Parser;
125
use derive_more::Display;

dylint.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[workspace.metadata.dylint]
22
libraries = [
3-
{ git = "https://github.com/KSXGitHub/perfectionist", tag = "0.0.0-rc.16" },
3+
{ git = "https://github.com/KSXGitHub/perfectionist", rev = "d793ada" },
44
]
55

66
["perfectionist::derive_ordering"]

src/args.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
#![cfg_attr(
2-
dylint_lib = "perfectionist",
3-
allow(
4-
perfectionist::lint_silence_reason,
5-
reason = "false positive on `clap`-derive-generated `#[allow]` attributes; see https://github.com/KSXGitHub/parallel-disk-usage/issues/430"
6-
)
7-
)]
8-
91
pub mod depth;
102
pub mod fraction;
113
pub mod quantity;

0 commit comments

Comments
 (0)