Removed unused imports and hid unused variables for the aura, balances, collator_selection, dmp_queue, proxy, society, xcmp queue pallet verifier migrations. Removed unused imports and hid unused variables authorship, parachain_system, transaction_payment, vesting pallet migrations.

sebastianmontero · sebastianmontero · commit fc63e2a3f907 · 2023-12-04T17:55:42.000-06:00
diff --git a/runtime/src/migrations/aura_verifier.rs b/runtime/src/migrations/aura_verifier.rs
@@ -1,11 +1,9 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
 use sp_consensus_aura::Slot;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -30,7 +28,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 		let authorities = Authorities::<Runtime>::get();
 
diff --git a/runtime/src/migrations/authorship.rs b/runtime/src/migrations/authorship.rs
@@ -1,12 +1,9 @@
-use super::*;
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade
 };
 
-#[cfg(feature = "try-runtime")]
-use sp_runtime::TryRuntimeError;
 
 /// The log target.
 const TARGET: &'static str = "runtime::authorship::migration";
diff --git a/runtime/src/migrations/balances_verifier.rs b/runtime/src/migrations/balances_verifier.rs
@@ -1,10 +1,8 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade
 };
-use frame_system::pallet_prelude::BlockNumberFor;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -21,7 +19,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 
 		let locks_count = pallet_balances::Locks::<Runtime>::iter_keys().count() as u32;
diff --git a/runtime/src/migrations/collator_selection_verifier.rs b/runtime/src/migrations/collator_selection_verifier.rs
@@ -1,10 +1,8 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -21,7 +19,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 
 		let authored_blocks_count = pallet_collator_selection::LastAuthoredBlock::<Runtime>::iter_keys().count() as u32;
diff --git a/runtime/src/migrations/dmp_queue_verifier.rs b/runtime/src/migrations/dmp_queue_verifier.rs
@@ -1,10 +1,9 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
-use super::*;
+
 use cumulus_pallet_dmp_queue::{ConfigData, PageIndexData};
 
 #[cfg(feature = "try-runtime")]
@@ -28,7 +27,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 
 		log::info!(target: TARGET, "Configuration: {:?}", Configuration::<Runtime>::get());
diff --git a/runtime/src/migrations/identity_verifier.rs b/runtime/src/migrations/identity_verifier.rs
@@ -3,7 +3,6 @@ use frame_support::{
 	pallet_prelude::*,
 	traits::{Currency, OnRuntimeUpgrade},
 };
-use frame_system::pallet_prelude::BlockNumberFor;
 use pallet_identity::*;
 
 #[cfg(feature = "try-runtime")]
@@ -54,7 +53,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In identity verifier post upgrade");
 		let vesting_version = StorageVersion::get::<Vesting>();
 		log::info!(
diff --git a/runtime/src/migrations/parachain_system.rs b/runtime/src/migrations/parachain_system.rs
@@ -1,12 +1,11 @@
-use super::*;
+
 use crate::*;
 use cumulus_pallet_parachain_system::MessagingStateSnapshot;
 use cumulus_primitives_core::{AbridgedHostConfiguration, PersistedValidationData};
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
 
@@ -112,7 +111,7 @@ pub mod v1 {
 		}
 
 		#[cfg(feature = "try-runtime")]
-		fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+		fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 			log::info!(target: TARGET, "In post upgrade, should be able to decode host configuration with the new structure");
 			let host_configuration = HostConfiguration::<Runtime>::get().unwrap();
 			log::info!("    ");
diff --git a/runtime/src/migrations/proxy_verifier.rs b/runtime/src/migrations/proxy_verifier.rs
@@ -1,10 +1,8 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -21,7 +19,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 
 		let proxies_count = pallet_proxy::Proxies::<Runtime>::iter_keys().count() as u32;
diff --git a/runtime/src/migrations/society_verifier.rs b/runtime/src/migrations/society_verifier.rs
@@ -1,10 +1,8 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -21,7 +19,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 
 		let candidates_count = pallet_society::Candidates::<Runtime>::iter_keys().count() as u32;
diff --git a/runtime/src/migrations/system.rs b/runtime/src/migrations/system.rs
@@ -1,11 +1,9 @@
-use super::*;
 use crate::*;
 use frame_system::EventRecord;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade,
 };
-use frame_system::pallet_prelude::BlockNumberFor;
 use frame_system::ConsumedWeight;
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -14,8 +12,8 @@ use sp_runtime::TryRuntimeError;
 const TARGET: &'static str = "runtime::system::migration";
 
 pub mod v0 {
-	use cumulus_primitives_core::relay_chain::AsyncBackingParams;
-	use sp_runtime::traits::Saturating;
+	// use cumulus_primitives_core::relay_chain::AsyncBackingParams;
+	// use sp_runtime::traits::Saturating;
 
 	use super::*;
 
@@ -95,7 +93,7 @@ pub mod v0 {
 		}
 
 		#[cfg(feature = "try-runtime")]
-		fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+		fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 			log::info!(target: TARGET, "In post upgrade");
 			let num_account_keys = frame_system::Account::<Runtime>::iter_keys().count() as u32;
 			let num_accounts = frame_system::Account::<Runtime>::iter_values().count() as u32;
diff --git a/runtime/src/migrations/transaction_payment.rs b/runtime/src/migrations/transaction_payment.rs
@@ -2,7 +2,7 @@ use frame_support::traits::OnRuntimeUpgrade;
 use codec::{Decode, Encode, MaxEncodedLen};
 use scale_info::TypeInfo;
 use sp_runtime::RuntimeDebug;
-use pallet_transaction_payment::{Multiplier, NextFeeMultiplierOnEmpty};
+use pallet_transaction_payment::Multiplier;
 use crate::*;
 
 #[cfg(feature = "try-runtime")]
@@ -67,7 +67,7 @@ use sp_runtime::traits::Saturating;
 		}
 
 		#[cfg(feature = "try-runtime")]
-		fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+		fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 			log::info!(target: TARGET, "In post upgrade");
 			let storage_version = StorageVersion::<Runtime>::get();
 			ensure!(storage_version == Releases::V2, "Storage version was not updated");
diff --git a/runtime/src/migrations/vesting.rs b/runtime/src/migrations/vesting.rs
@@ -87,7 +87,7 @@ use sp_runtime::traits::Saturating;
 		}
 
 		#[cfg(feature = "try-runtime")]
-		fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+		fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 			let result = pallet_vesting::migrations::v1::post_migrate::<Runtime>();
 			match result {
 				Ok(_) => {
diff --git a/runtime/src/migrations/xcmp_queue_verifier.rs b/runtime/src/migrations/xcmp_queue_verifier.rs
@@ -1,13 +1,11 @@
 use crate::*;
 use frame_support::{
 	pallet_prelude::*,
-	traits::{Currency, OnRuntimeUpgrade},
+	traits::OnRuntimeUpgrade
 };
-use frame_system::pallet_prelude::BlockNumberFor;
 use cumulus_pallet_xcmp_queue::{QueueConfigData, OverweightIndex};
 use cumulus_primitives_core::relay_chain::BlockNumber as RelayBlockNumber;
 use crate::ParaId;
-use super::*;
 
 #[cfg(feature = "try-runtime")]
 use sp_runtime::TryRuntimeError;
@@ -32,7 +30,7 @@ impl OnRuntimeUpgrade for Migrate {
 	}
 
 	#[cfg(feature = "try-runtime")]
-	fn post_upgrade(state: Vec<u8>) -> Result<(), TryRuntimeError> {
+	fn post_upgrade(_state: Vec<u8>) -> Result<(), TryRuntimeError> {
 		log::info!(target: TARGET, "In post upgrade");
 		let overweight_tracked_count = Overweight::<Runtime>::count();
 		let overweight_count = Overweight::<Runtime>::iter_keys().count() as u32;
