warning: useless conversion to the same type: `pallet::Event<T, I>` --> pallets/pass/src/lib.rs:290:13 | 290 | / Event::<T, I>::SessionCreated { 291 | | session_key: session_key.clone(), 292 | | until, 293 | | } 294 | | .into(), | |___________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion help: consider removing `.into()` | 290 ~ Event::<T, I>::SessionCreated { 291 + session_key: session_key.clone(), 292 + until, 293 ~ }, |

warning: using `clone` on type `[u8; 32]` which implements the `Copy` trait --> pallets/pass/src/lib.rs:236:28 | 236 | device_id: device_id.clone(), | ^^^^^^^^^^^^^^^^^ help: try dereferencing it: `*device_id` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy = note: `#[warn(clippy::clone_on_copy)]` on by default

warning: useless conversion to the same type: `pallet::Event<T, I>` --> pallets/pass/src/lib.rs:234:13 | 234 | / Event::<T, I>::AddedDevice { 235 | | who: who.clone(), 236 | | device_id: device_id.clone(), 237 | | } 238 | | .into(), | |___________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default help: consider removing `.into()` | 234 ~ Event::<T, I>::AddedDevice { 235 + who: who.clone(), 236 + device_id: device_id.clone(), 237 ~ }, |

warning: using `map_err` over `inspect_err` --> pallets/pass/src/lib.rs:67:15 | 67 | #[pallet::pallet] | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_inspect = note: `#[warn(clippy::manual_inspect)]` on by default help: try | 67 - #[pallet::pallet] 67 + #[pallet::&inspect_err] |

warning: the borrowed expression implements the required traits --> pallets/pass/src/lib.rs:153:60 | 153 | let device = Devices::<T, I>::get(&account_id, &device_id) | ^^^^^^^^^^ help: change this to: `device_id` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default

warning: deref on an immutable reference --> pallets/pass/src/benchmarking.rs:41:33 | 41 | let user_id = hash::<T>(&*b"my-account"); | ^^^^^^^^^^^^^^^ help: if you would like to reborrow, try removing `&*`: `b"my-account"` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#borrow_deref_ref = note: `#[warn(clippy::borrow_deref_ref)]` on by default

warning: duplicated attribute --> pallets/pass/src/benchmarking.rs:1:8 | 1 | #![cfg(feature = "runtime-benchmarks")] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | note: first defined here --> pallets/pass/src/lib.rs:19:7 | 19 | #[cfg(feature = "runtime-benchmarks")] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove this attribute --> pallets/pass/src/benchmarking.rs:1:8 | 1 | #![cfg(feature = "runtime-benchmarks")] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#duplicated_attributes = note: `#[warn(clippy::duplicated_attributes)]` on by default

warning: bound is defined in more than one place --> pallets/pass/src/benchmarking.rs:28:1 | 28 | / #[instance_benchmarks( 29 | | where 30 | | T: frame_system::Config + crate::Config<I>, | | ^ 31 | | T::Hash: Into<HashedUserId>, 32 | | RuntimeEventFor<T, I>: From<frame_system::Event<T>>, 33 | | )] | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#multiple_bound_locations = note: `#[warn(clippy::multiple_bound_locations)]` on by default = note: this warning originates in the attribute macro `instance_benchmarks` (in Nightly builds, run with -Z macro-backtrace for more info)

warning: `self.0.saturating_sub(n.get())` is never smaller than `Self::MIN.0` and has therefore no effect --> traits/memberships/src/lib.rs:98:14 | 98 | Self(self.0.saturating_sub(n.get()).max(Self::MIN.0)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `self.0.saturating_sub(n.get())` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_min_or_max = note: `#[warn(clippy::unnecessary_min_or_max)]` on by default

warning: using `map_err` over `inspect_err` --> pallets/template/src/lib.rs:34:15 | 34 | #[pallet::pallet] | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_inspect = note: `#[warn(clippy::manual_inspect)]` on by default help: try | 34 - #[pallet::pallet] 34 + #[pallet::&inspect_err] |

warning: using `map_err` over `inspect_err` --> pallets/gas-transaction-payment/src/lib.rs:32:15 | 32 | #[pallet::pallet] | ^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_inspect = note: `#[warn(clippy::manual_inspect)]` on by default help: try | 32 - #[pallet::pallet] 32 + #[pallet::&inspect_err] |

warning: empty doc comment --> pallets/referenda-tracks/src/lib.rs:86:9 | 86 | /// | ^^^ | = help: consider removing or filling it = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_docs = note: `#[warn(clippy::empty_docs)]` on by default

warning: useless conversion to the same type: `()` --> pallets/referenda-tracks/src/impls.rs:96:12 | 96 | Ok(().into()) | ^^^^^^^^^ help: consider removing `.into()`: `()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default

warning: passing a unit value to a function --> pallets/referenda-tracks/src/impls.rs:96:9 | 96 | Ok(().into()) | ^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unit_arg = note: `#[warn(clippy::unit_arg)]` on by default help: move the expression in front of the call and replace it with the unit literal `()` | 96 ~ ().into(); 97 + Ok(()) |

warning: using `clone` on type `DispatchInfo` which implements the `Copy` trait --> pallets/gas-transaction-payment/src/extensions.rs:120:22 | 120 | &info.clone().into(), | ^^^^^^^^^^^^ help: try dereferencing it: `(*info)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy = note: `#[warn(clippy::clone_on_copy)]` on by default

warning: useless conversion to the same type: `frame_support::dispatch::DispatchInfo` --> pallets/gas-transaction-payment/src/extensions.rs:120:22 | 120 | &info.clone().into(), | ^^^^^^^^^^^^^^^^^^^ help: consider removing `.into()`: `info.clone()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default

warning: redundant pattern matching, consider using `is_some()` --> pallets/gas-transaction-payment/src/extensions.rs:102:16 | 102 | if let Some(_) = T::GasBurner::check_available_gas(who, &info.weight) { | -------^^^^^^^------------------------------------------------------- help: try: `if T::GasBurner::check_available_gas(who, &info.weight).is_some()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching = note: `#[warn(clippy::redundant_pattern_matching)]` on by default

warning: binary comparison to literal `Option::None` --> pallets/referenda-tracks/src/impls.rs:41:13 | 41 | Tracks::<T, I>::get(id) == None, | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `Option::is_none()` instead: `Tracks::<T, I>::get(id).is_none()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#partialeq_to_none = note: `#[warn(clippy::partialeq_to_none)]` on by default

warning: using `clone` on type `<T as Config<I>>::TrackId` which implements the `Copy` trait --> pallets/referenda-tracks/src/benchmarking.rs:86:57 | 86 | OriginToTrackId::<T, I>::insert(origin.clone(), id.clone()); | ^^^^^^^^^^ help: try dereferencing it: `*id` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy

warning: using `clone` on type `<T as Config<I>>::TrackId` which implements the `Copy` trait --> pallets/referenda-tracks/src/benchmarking.rs:85:32 | 85 | Tracks::<T, I>::insert(id.clone(), track.clone()); | ^^^^^^^^^^ help: try dereferencing it: `*id` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy = note: `#[warn(clippy::clone_on_copy)]` on by default

The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/