feat(pop-api): errors #17
Annotations
2 errors and 14 warnings
check
Process completed with exit code 101.
|
test
Process completed with exit code 101.
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
this function has too many arguments (14/7):
node/src/service.rs#L368
warning: this function has too many arguments (14/7)
--> node/src/service.rs:368:1
|
368 | / fn start_consensus(
369 | | client: Arc<ParachainClient>,
370 | | block_import: ParachainBlockImport,
371 | | prometheus_registry: Option<&Registry>,
... |
382 | | announce_block: Arc<dyn Fn(Hash, Option<Vec<u8>>) + Send + Sync>,
383 | | ) -> Result<(), sc_service::Error> {
| |__________________________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: `#[warn(clippy::too_many_arguments)]` on by default
|
large size difference between variants:
node/src/cli.rs#L5
warning: large size difference between variants
--> node/src/cli.rs:5:1
|
5 | / pub enum Subcommand {
6 | | /// Build a chain specification.
7 | | BuildSpec(sc_cli::BuildSpecCmd),
8 | |
... |
19 | | ImportBlocks(sc_cli::ImportBlocksCmd),
| | ------------------------------------- the second-largest variant contains at least 240 bytes
... |
39 | | Benchmark(frame_benchmarking_cli::BenchmarkCmd),
| | ----------------------------------------------- the largest variant contains at least 512 bytes
... |
48 | | Key(sc_cli::KeySubcommand),
49 | | }
| |_^ the entire enum is at least 512 bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#large_enum_variant
= note: `#[warn(clippy::large_enum_variant)]` on by default
help: consider boxing the large fields to reduce the total size of the enum
|
39 | Benchmark(Box<frame_benchmarking_cli::BenchmarkCmd>),
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
function `convert_err` is never used:
runtime/src/extensions/pop_api_extension.rs#L19
warning: function `convert_err` is never used
--> runtime/src/extensions/pop_api_extension.rs:19:4
|
19 | fn convert_err(err_msg: &'static str) -> impl FnOnce(DispatchError) -> DispatchError {
| ^^^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
unreachable pattern:
runtime/src/extensions/ext_impl/read_state.rs#L30
warning: unreachable pattern
--> runtime/src/extensions/ext_impl/read_state.rs:30:9
|
30 | _ => Err(DispatchError::Other("Unable to read provided key.")),
| ^
|
= note: `#[warn(unreachable_patterns)]` on by default
|
this is a decimal constant:
runtime/src/lib.rs#L191
warning: this is a decimal constant
--> runtime/src/lib.rs:191:19
|
191 | spec_version: 00_01_00,
| ^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#zero_prefixed_literal
= note: `#[warn(clippy::zero_prefixed_literal)]` on by default
help: if you mean to use a decimal constant, remove the `0` to avoid confusion
|
191 | spec_version: 1_00,
| ~~~~
help: if you mean to use an octal constant, use `0o`
|
191 | spec_version: 0o1_00,
| ~~~~~~
|
type alias `MaxTips` is never used:
pop-api/src/lib.rs#L25
warning: type alias `MaxTips` is never used
--> pop-api/src/lib.rs:25:6
|
25 | type MaxTips = u32;
| ^^^^^^^
|
type alias `KeyLimit` is never used:
pop-api/src/lib.rs#L24
warning: type alias `KeyLimit` is never used
--> pop-api/src/lib.rs:24:6
|
24 | type KeyLimit = u32;
| ^^^^^^^^
|
type alias `Signature` is never used:
pop-api/src/lib.rs#L22
warning: type alias `Signature` is never used
--> pop-api/src/lib.rs:22:6
|
22 | type Signature = MultiSignature;
| ^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
this import is redundant:
pop-api/src/lib.rs#L8
warning: this import is redundant
--> pop-api/src/lib.rs:8:1
|
8 | use scale;
| ^^^^^^^^^^ help: remove it entirely
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_component_path_imports
= note: `#[warn(clippy::single_component_path_imports)]` on by default
|
clippy
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/cache@v3, actions-rs/clippy-check@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
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/
|
check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/cache@v3, Swatinem/rust-cache@v2.5.0. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|