chore: remove unused_assignments from allow list

viz-rs · Dec 16, 2023 · db53a4d · db53a4d
1 parent 2c8c8ff
commit db53a4d
Show file tree

Hide file tree

Showing 8 changed files with 15 additions and 26 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -126,6 +126,7 @@ missing_docs = "warn"
 [workspace.lints.clippy]
 all = "deny"
 pedantic = "deny"
+needless_pass_by_value = "warn"
 module_name_repetitions = { level = "allow", priority = 1 }
 too_many_lines = { level = "allow", priority = 1 }
 type_complexity = { level = "allow", priority = 1 }
diff --git a/viz-core/src/lib.rs b/viz-core/src/lib.rs
@@ -6,10 +6,7 @@
 #![doc(html_favicon_url = "https://viz.rs/logo.svg")]
 #![doc(test(
     no_crate_inject,
-    attr(
-        deny(warnings, rust_2018_idioms),
-        allow(dead_code, unused_assignments, unused_variables)
-    )
+    attr(deny(warnings, rust_2018_idioms), allow(dead_code, unused_variables))
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]
 

diff --git a/viz-handlers/src/lib.rs b/viz-handlers/src/lib.rs
@@ -4,10 +4,7 @@
 #![doc(html_favicon_url = "https://viz.rs/logo.svg")]
 #![doc(test(
     no_crate_inject,
-    attr(
-        deny(warnings, rust_2018_idioms),
-        allow(dead_code, unused_assignments, unused_variables)
-    )
+    attr(deny(warnings, rust_2018_idioms), allow(dead_code, unused_variables))
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]
 

diff --git a/viz-handlers/src/prometheus.rs b/viz-handlers/src/prometheus.rs
@@ -24,7 +24,7 @@ pub struct Prometheus {
 }
 
 impl Prometheus {
-    /// Creates a new Prometheus.
+    /// Creates a new [`Prometheus`].
     #[must_use]
     pub fn new(registry: Registry) -> Self {
         Self { registry }

diff --git a/viz-macros/src/lib.rs b/viz-macros/src/lib.rs
@@ -29,10 +29,7 @@
 #![doc(html_favicon_url = "https://viz.rs/logo.svg")]
 #![doc(test(
     no_crate_inject,
-    attr(
-        deny(warnings, rust_2018_idioms),
-        allow(dead_code, unused_assignments, unused_variables)
-    )
+    attr(deny(warnings, rust_2018_idioms), allow(dead_code, unused_variables))
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]
 

diff --git a/viz-router/src/lib.rs b/viz-router/src/lib.rs
@@ -4,10 +4,7 @@
 #![doc(html_favicon_url = "https://viz.rs/logo.svg")]
 #![doc(test(
     no_crate_inject,
-    attr(
-        deny(warnings, rust_2018_idioms),
-        allow(dead_code, unused_assignments, unused_variables)
-    )
+    attr(deny(warnings, rust_2018_idioms), allow(dead_code, unused_variables))
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]
 #![allow(clippy::needless_pass_by_value)]
@@ -16,12 +13,15 @@
 pub(crate) mod macros;
 
 mod resources;
+pub use resources::Resources;
+
 mod route;
+pub use route::*;
+
 mod router;
+pub use router::Router;
+
 mod tree;
+pub use tree::Tree;
 
 pub use path_tree::{Path, PathTree};
-pub use resources::Resources;
-pub use route::*;
-pub use router::Router;
-pub use tree::Tree;
diff --git a/viz-router/src/route.rs b/viz-router/src/route.rs
@@ -167,7 +167,7 @@ impl FromIterator<(Method, BoxHandler)> for Route {
 pub fn on<H, O>(method: Method, handler: H) -> Route
 where
     H: Handler<Request, Output = Result<O>> + Clone,
-    O: IntoResponse + Send + Sync + 'static,
+    O: IntoResponse + Send + 'static,
 {
     Route::new().on(method, handler)
 }

diff --git a/viz/src/lib.rs b/viz/src/lib.rs
@@ -517,10 +517,7 @@
 #![doc(html_favicon_url = "https://viz.rs/logo.svg")]
 #![doc(test(
     no_crate_inject,
-    attr(
-        deny(warnings, rust_2018_idioms),
-        allow(dead_code, unused_assignments, unused_variables)
-    )
+    attr(deny(warnings, rust_2018_idioms), allow(dead_code, unused_variables))
 ))]
 #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]