al3mart/query_state & pop_api_primitives #12
Annotations
2 errors and 21 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#L31
warning: unreachable pattern
--> runtime/src/extensions/ext_impl/read_state.rs:31:9
|
31 | _ => 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,
| ~~~~~~
|
unused import: `codec::Decode`:
runtime/src/extensions/ext_impl/read_state.rs#L1
warning: unused import: `codec::Decode`
--> runtime/src/extensions/ext_impl/read_state.rs:1:5
|
1 | use codec::Decode;
| ^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
|
multiple variants are never constructed:
pop-api/src/v0/nfts.rs#L28
warning: multiple variants are never constructed
--> pop-api/src/v0/nfts.rs:28:5
|
21 | pub(crate) enum NftCalls {
| -------- variants in this enum
...
28 | Destroy { collection: CollectionId },
| ^^^^^^^
...
37 | Burn {
| ^^^^
...
42 | Transfer {
| ^^^^^^^^
...
48 | Redeposit {
| ^^^^^^^^^
...
53 | LockItemTransfer {
| ^^^^^^^^^^^^^^^^
...
58 | UnlockItemTransfer {
| ^^^^^^^^^^^^^^^^^^
...
68 | TransferOwnership {
| ^^^^^^^^^^^^^^^^^
...
73 | SetTeam {
| ^^^^^^^
...
80 | ApproveTransfer {
| ^^^^^^^^^^^^^^^
...
87 | CancelApproval {
| ^^^^^^^^^^^^^^
...
93 | ClearAllTransferApprovals {
| ^^^^^^^^^^^^^^^^^^^^^^^^^
...
98 | LockItemProperties {
| ^^^^^^^^^^^^^^^^^^
...
120 | ApproveItemAttribute {
| ^^^^^^^^^^^^^^^^^^^^
...
126 | CancelItemAttributesApproval {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...
132 | SetMetadata {
| ^^^^^^^^^^^
...
138 | ClearMetadata {
| ^^^^^^^^^^^^^
...
143 | SetCollectionMetadata {
| ^^^^^^^^^^^^^^^^^^^^^
...
148 | ClearCollectionMetadata { collection: CollectionId },
| ^^^^^^^^^^^^^^^^^^^^^^^
149 | #[codec(index = 28)]
150 | SetAcceptOwnership {
| ^^^^^^^^^^^^^^^^^^
...
155 | SetCollectionMaxSupply {
| ^^^^^^^^^^^^^^^^^^^^^^
...
165 | Price {
| ^^^^^
...
171 | BuyItem {
| ^^^^^^^
...
190 | CancelSwap {
| ^^^^^^^^^^
...
195 | ClaimSwap {
| ^^^^^^^^^
|
field `nft` is never read:
pop-api/src/impls/pop_network.rs#L29
warning: field `nft` is never read
--> pop-api/src/impls/pop_network.rs:29:5
|
28 | pub struct Api {
| --- field in this struct
29 | nft: Nfts,
| ^^^
|
type alias `MaxTips` is never used:
pop-api/src/impls/pop_network.rs#L22
warning: type alias `MaxTips` is never used
--> pop-api/src/impls/pop_network.rs:22:6
|
22 | type MaxTips = u32;
| ^^^^^^^
|
type alias `KeyLimit` is never used:
pop-api/src/impls/pop_network.rs#L21
warning: type alias `KeyLimit` is never used
--> pop-api/src/impls/pop_network.rs:21:6
|
21 | type KeyLimit = u32;
| ^^^^^^^^
|
type alias `Signature` is never used:
pop-api/src/impls/pop_network.rs#L18
warning: type alias `Signature` is never used
--> pop-api/src/impls/pop_network.rs:18:6
|
18 | type Signature = MultiSignature;
| ^^^^^^^^^
|
type alias `MaxTips` is never used:
pop-api/src/lib.rs#L27
warning: type alias `MaxTips` is never used
--> pop-api/src/lib.rs:27:6
|
27 | type MaxTips = u32;
| ^^^^^^^
|
type alias `KeyLimit` is never used:
pop-api/src/lib.rs#L26
warning: type alias `KeyLimit` is never used
--> pop-api/src/lib.rs:26:6
|
26 | type KeyLimit = u32;
| ^^^^^^^^
|
type alias `Signature` is never used:
pop-api/src/lib.rs#L24
warning: type alias `Signature` is never used
--> pop-api/src/lib.rs:24:6
|
24 | type Signature = MultiSignature;
| ^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
this import is redundant:
pop-api/src/lib.rs#L10
warning: this import is redundant
--> pop-api/src/lib.rs:10:1
|
10 | 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
|
unused import: `pop_api_primitives::storage_keys`:
pop-api/src/lib.rs#L6
warning: unused import: `pop_api_primitives::storage_keys`
--> pop-api/src/lib.rs:6:5
|
6 | use pop_api_primitives::storage_keys;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on 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/.
|
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/
|
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/.
|