Przeglądaj źródła

Change the way clippy is invoked

The touch command means it always prints its warnings, and the per-crate allows more fine-grained control over where warnings are switched off, without having to scatter them all through the code.
Benjamin Sago 4 lat temu
rodzic
commit
83ff09f610
6 zmienionych plików z 20 dodań i 6 usunięć
  1. 1 1
      Cargo.toml
  2. 2 4
      Justfile
  3. 0 1
      clippy.toml
  4. 5 0
      dns-transport/src/lib.rs
  5. 7 0
      dns/src/lib.rs
  6. 5 0
      src/main.rs

+ 1 - 1
Cargo.toml

@@ -5,7 +5,7 @@ authors = ["Benjamin Sago <ogham@bsago.me>"]
 edition = "2018"
 exclude = [
     "/completions/*", "/man/*", "/xtests/*",
-    "/clippy.toml", "/dog-screenshot.png", "/Justfile", "/README.md",
+    "/dog-screenshot.png", "/Justfile", "/README.md", "/.rustfmt.toml", "/.travis.yml",
 ]
 
 [[bin]]

+ 2 - 4
Justfile

@@ -40,7 +40,5 @@ export DOG_DEBUG := ""
 
 # lints the code
 @clippy:
-    cargo clippy -- -A clippy::module_name_repetitions \
-                    -A clippy::module_inception \
-                    -A clippy::non_ascii_literal \
-                    -A clippy::use_self
+    touch dns/src/lib.rs
+    cargo clippy

+ 0 - 1
clippy.toml

@@ -1 +0,0 @@
-

+ 5 - 0
dns-transport/src/lib.rs

@@ -11,6 +11,11 @@
 #![warn(trivial_casts, trivial_numeric_casts)]
 #![warn(unused)]
 
+#![warn(clippy::all, clippy::pedantic)]
+#![allow(clippy::module_name_repetitions)]
+#![allow(clippy::pub_enum_variant_names)]
+#![allow(clippy::wildcard_imports)]
+
 #![deny(unsafe_code)]
 
 use async_trait::async_trait;

+ 7 - 0
dns/src/lib.rs

@@ -9,6 +9,13 @@
 #![warn(trivial_casts, trivial_numeric_casts)]
 #![warn(unused)]
 
+#![warn(clippy::all, clippy::pedantic)]
+#![allow(clippy::missing_errors_doc)]
+#![allow(clippy::module_name_repetitions)]
+#![allow(clippy::must_use_candidate)]
+#![allow(clippy::struct_excessive_bools)]
+#![allow(clippy::wildcard_imports)]
+
 #![deny(unsafe_code)]
 
 

+ 5 - 0
src/main.rs

@@ -11,6 +11,11 @@
 #![warn(trivial_casts, trivial_numeric_casts)]
 #![warn(unused)]
 
+#![warn(clippy::all, clippy::pedantic)]
+#![allow(clippy::enum_glob_use)]
+#![allow(clippy::module_name_repetitions)]
+#![allow(clippy::wildcard_imports)]
+
 #![deny(unsafe_code)]