Remove a few unnecessary lint suppressions

autonomys · Sep 26, 2024 · b9cfc12 · b9cfc12
1 parent d96e852
commit b9cfc12
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 13 deletions.
diff --git a/crates/pallet-domains/src/lib.rs b/crates/pallet-domains/src/lib.rs
@@ -171,8 +171,6 @@ pub(crate) type StateRootOf<T> = <<T as frame_system::Config>::Hashing as Hash>:
 
 #[frame_support::pallet]
 mod pallet {
-    #![allow(clippy::large_enum_variant)]
-
     #[cfg(not(feature = "runtime-benchmarks"))]
     use crate::block_tree::AcceptedReceiptType;
     use crate::block_tree::{

diff --git a/crates/pallet-rewards/src/mock.rs b/crates/pallet-rewards/src/mock.rs
@@ -1,5 +1,3 @@
-#![allow(missing_debug_implementations)]
-
 use frame_support::derive_impl;
 use frame_support::traits::{ConstU128, ConstU32};
 use sp_runtime::traits::parameter_types;

diff --git a/crates/subspace-runtime/src/lib.rs b/crates/subspace-runtime/src/lib.rs
@@ -18,9 +18,11 @@
 #![feature(const_option, const_trait_impl, variant_count)]
 // `construct_runtime!` does a lot of recursion and requires us to increase the limit to 256.
 #![recursion_limit = "256"]
-// Silence a rust-analyzer warning in `construct_runtime!`. This warning isn't present in rustc output.
-// TODO: remove when upstream issue is fixed: <https://github.com/rust-lang/rust-analyzer/issues/16514>
-#![allow(non_camel_case_types)]
+// TODO: remove when upstream issue is fixed
+#![allow(
+    non_camel_case_types,
+    reason = "https://github.com/rust-lang/rust-analyzer/issues/16514"
+)]
 
 mod domains;
 mod fees;

diff --git a/domains/pallets/executive/src/mock.rs b/domains/pallets/executive/src/mock.rs
@@ -1,6 +1,8 @@
-// Silence a rust-analyzer warning in `construct_runtime!`. This warning isn't present in rustc output.
-// TODO: remove when upstream issue is fixed: <https://github.com/rust-lang/rust-analyzer/issues/16514>
-#![allow(non_camel_case_types)]
+// TODO: remove when upstream issue is fixed
+#![allow(
+    non_camel_case_types,
+    reason = "https://github.com/rust-lang/rust-analyzer/issues/16514"
+)]
 
 use crate as pallet_executive;
 use crate::Config;

diff --git a/test/subspace-test-runtime/src/lib.rs b/test/subspace-test-runtime/src/lib.rs
@@ -18,9 +18,11 @@
 #![feature(const_option, variant_count)]
 // `construct_runtime!` does a lot of recursion and requires us to increase the limit to 256.
 #![recursion_limit = "256"]
-// Silence a rust-analyzer warning in `construct_runtime!`. This warning isn't present in rustc output.
-// TODO: remove when upstream issue is fixed: <https://github.com/rust-lang/rust-analyzer/issues/16514>
-#![allow(non_camel_case_types)]
+// TODO: remove when upstream issue is fixed
+#![allow(
+    non_camel_case_types,
+    reason = "https://github.com/rust-lang/rust-analyzer/issues/16514"
+)]
 
 // Make the WASM binary available.
 #[cfg(feature = "std")]