Skip to content

Pallet Pass

Pallet Pass #3

Triggered via pull request September 26, 2024 17:18
@olanodolanod
synchronize #22
pallet-pass
Status Failure
Total duration 1m 29s
Artifacts

ci.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 16 warnings
the trait bound `util::Dev<T, A, Ch, Cred>: parity_scale_codec::MaxEncodedLen` is not satisfied: traits/authn/src/util.rs#L36
error[E0277]: the trait bound `util::Dev<T, A, Ch, Cred>: parity_scale_codec::MaxEncodedLen` is not satisfied --> traits/authn/src/util.rs:36:44 | 36 | impl<T, A, Ch, Cred> UserAuthenticator for Dev<T, A, Ch, Cred> | ^^^^^^^^^^^^^^^^^^^ the trait `parity_scale_codec::MaxEncodedLen` is not implemented for `util::Dev<T, A, Ch, Cred>` | = help: the following other types implement trait `parity_scale_codec::MaxEncodedLen`: () (TupleElement0, TupleElement1) (TupleElement0, TupleElement1, TupleElement2) (TupleElement0, TupleElement1, TupleElement2, TupleElement3) (TupleElement0, TupleElement1, TupleElement2, TupleElement3, TupleElement4) (TupleElement0, TupleElement1, TupleElement2, TupleElement3, TupleElement4, TupleElement5) (TupleElement0, TupleElement1, TupleElement2, TupleElement3, TupleElement4, TupleElement5, TupleElement6) (TupleElement0, TupleElement1, TupleElement2, TupleElement3, TupleElement4, TupleElement5, TupleElement6, TupleElement7) and 106 others note: required by a bound in `UserAuthenticator` --> traits/authn/src/lib.rs:51:42 | 51 | pub trait UserAuthenticator: FullCodec + MaxEncodedLen + TypeInfo { | ^^^^^^^^^^^^^ required by this bound in `UserAuthenticator`
test
Process completed with exit code 101.
clippy
Clippy had exited with the 101 exit code
check
Process completed with exit code 101.
using `map_err` over `inspect_err`: pallets/gas-transaction-payment/src/lib.rs#L32
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] |
using `map_err` over `inspect_err`: pallets/referenda-tracks/src/lib.rs#L93
warning: using `map_err` over `inspect_err` --> pallets/referenda-tracks/src/lib.rs:93:15 | 93 | #[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 | 93 - #[pallet::pallet] 93 + #[pallet::&inspect_err] |
`self.0.saturating_sub(n.get())` is never smaller than `Self::MIN.0` and has therefore no effect: traits/memberships/src/lib.rs#L98
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
using `clone` on type `DispatchInfo` which implements the `Copy` trait: pallets/gas-transaction-payment/src/extensions.rs#L120
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
useless conversion to the same type: `frame_support::dispatch::DispatchInfo`: pallets/gas-transaction-payment/src/extensions.rs#L120
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
redundant pattern matching, consider using `is_some()`: pallets/gas-transaction-payment/src/extensions.rs#L102
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
empty doc comment: pallets/referenda-tracks/src/lib.rs#L86
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
useless conversion to the same type: `()`: pallets/referenda-tracks/src/impls.rs#L96
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
passing a unit value to a function: pallets/referenda-tracks/src/impls.rs#L96
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(()) |
binary comparison to literal `Option::None`: pallets/referenda-tracks/src/impls.rs#L41
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
using `clone` on type `<T as Config<I>>::TrackId` which implements the `Copy` trait: pallets/referenda-tracks/src/benchmarking.rs#L86
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
using `clone` on type `<T as Config<I>>::TrackId` which implements the `Copy` trait: pallets/referenda-tracks/src/benchmarking.rs#L85
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
duplicated attribute: pallets/referenda-tracks/src/benchmarking.rs#L20
warning: duplicated attribute --> pallets/referenda-tracks/src/benchmarking.rs:20:8 | 20 | #![cfg(feature = "runtime-benchmarks")] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | note: first defined here --> pallets/referenda-tracks/src/lib.rs:37:7 | 37 | #[cfg(feature = "runtime-benchmarks")] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove this attribute --> pallets/referenda-tracks/src/benchmarking.rs:20:8 | 20 | #![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
unused import: `sp_runtime::traits::Morph`: traits/authn/src/util.rs#L4
warning: unused import: `sp_runtime::traits::Morph` --> traits/authn/src/util.rs:4:21 | 4 | use frame_support::{sp_runtime::traits::Morph, traits::Get}; | ^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
clippy
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/
clippy
The following actions use a deprecated Node.js version and will be forced to run on node20: actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/