Skip to content

Bump async-trait from 0.1.82 to 0.1.83 (#49) #24

Bump async-trait from 0.1.82 to 0.1.83 (#49)

Bump async-trait from 0.1.82 to 0.1.83 (#49) #24

Triggered via push November 5, 2024 15:46
Status Success
Total duration 1m 40s
Artifacts

clippy.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
clippy
clippy-action doesn't have permissions to create Check Runs, disabling!
clippy
Resource not accessible by integration - https://docs.github.com/rest/checks/runs#create-a-check-run
use of `expect` followed by a function call: src/lib.rs#L58
warning: use of `expect` followed by a function call --> src/config_loader.rs:58:28 | 58 | env::var(env_name).expect(&format!("Password {} not loaded in memory", env_name)), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `unwrap_or_else(|_| panic!("Password {} not loaded in memory", env_name))` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call = note: `#[warn(clippy::expect_fun_call)]` on by default
deref which would be done by auto-deref: src/lib.rs#L12
warning: deref which would be done by auto-deref --> src/importer.rs:12:35 | 12 | let uri = SecretUri::from_str(&*seed).expect("valid URI"); | ^^^^^^ help: try: `&seed` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref = note: `#[warn(clippy::explicit_auto_deref)]` on by default
passing a unit value to a function: src/lib.rs#L43
warning: passing a unit value to a function --> src/transaction.rs:43:9 | 43 | Ok(out.extend_from_slice(&self.0)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = 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 `()` | 43 ~ out.extend_from_slice(&self.0); 44 + Ok(()) |
this function has too many arguments (9/7): src/lib.rs#L244
warning: this function has too many arguments (9/7) --> src/transaction.rs:244:5 | 244 | / async fn submit_and_watch( 245 | | platform_client: Client, 246 | | platform_url: String, 247 | | platform_token: String, ... | 253 | | block_header: SubstrateHeader<u32, BlakeTwo256>, 254 | | ) -> Result<String, Box<dyn std::error::Error + Send + Sync>> { | |_________________________________________________________________^ | = 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
the borrowed expression implements the required traits: src/lib.rs#L256
warning: the borrowed expression implements the required traits --> src/transaction.rs:256:38 | 256 | let public_key = hex::encode(&keypair.public_key().0); | ^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `keypair.public_key().0` | = 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
the borrowed expression implements the required traits: src/lib.rs#L286
warning: the borrowed expression implements the required traits --> src/transaction.rs:286:38 | 286 | let encoded_tx = hex::encode(&signed_tx.encoded()); | ^^^^^^^^^^^^^^^^^^^^ help: change this to: `signed_tx.encoded()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
this function has too many arguments (8/7): src/lib.rs#L365
warning: this function has too many arguments (8/7) --> src/transaction.rs:365:5 | 365 | / async fn transaction_handler( 366 | | chain_client: Arc<OnlineClient<PolkadotConfig>>, 367 | | platform_client: Client, 368 | | block_subscription: Arc<SubscriptionParams>, ... | 378 | | }: TransactionRequest, 379 | | ) { | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
7 warnings emitted
warning: 7 warnings emitted