Skip to content

Commit b205459

Browse files
committed
Extract deny lints to const array
1 parent 29b67a6 commit b205459

File tree

1 file changed

+19
-24
lines changed

1 file changed

+19
-24
lines changed

tests/dogfood.rs

Lines changed: 19 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,20 @@ fn dogfood() {
5757
);
5858
}
5959

60+
const DENY_LINTS: &[&str] = &[
61+
"unfulfilled_lint_expectations",
62+
"clippy::all",
63+
"clippy::pedantic",
64+
"clippy::dbg_macro",
65+
"clippy::decimal_literal_representation",
66+
"clippy::derive_partial_eq_without_eq",
67+
"clippy::iter_on_single_items",
68+
"clippy::needless_pass_by_ref_mut",
69+
"clippy::significant_drop_tightening",
70+
"clippy::tuple_array_conversions",
71+
"clippy::useless_let_if_seq",
72+
];
73+
6074
#[must_use]
6175
fn run_clippy_for_package(project: &str) -> bool {
6276
let root_dir = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
@@ -82,30 +96,11 @@ fn run_clippy_for_package(project: &str) -> bool {
8296

8397
command.arg("--");
8498
command.arg("-Cdebuginfo=0"); // disable debuginfo to generate less data in the target dir
85-
command.args([
86-
"-D",
87-
"unfulfilled_lint_expectations",
88-
"-D",
89-
"clippy::all",
90-
"-D",
91-
"clippy::pedantic",
92-
"-D",
93-
"clippy::dbg_macro",
94-
"-D",
95-
"clippy::decimal_literal_representation",
96-
"-D",
97-
"clippy::derive_partial_eq_without_eq",
98-
"-D",
99-
"clippy::iter_on_single_items",
100-
"-D",
101-
"clippy::needless_pass_by_ref_mut",
102-
"-D",
103-
"clippy::significant_drop_tightening",
104-
"-D",
105-
"clippy::tuple_array_conversions",
106-
"-D",
107-
"clippy::useless_let_if_seq",
108-
]);
99+
100+
for lint in DENY_LINTS {
101+
command.args(["-D", lint]);
102+
}
103+
109104
if !cfg!(feature = "internal") {
110105
// running a clippy built without internal lints on the clippy source
111106
// that contains e.g. `allow(clippy::symbol_as_str)`

0 commit comments

Comments
 (0)