Daemon Sender separation #1576
test.yml
on: pull_request
ubuntu / stable / coverage
4m 35s
Matrix: doc-tests / stable
Matrix: required
Annotations
24 errors
this expression creates a reference which is immediately dereferenced by the compiler:
cw-orch-daemon/src/live_mock.rs#L85
error: this expression creates a reference which is immediately dereferenced by the compiler
--> cw-orch-daemon/src/live_mock.rs:85:82
|
85 | let querier = CosmWasm::<Wallet>::new_sync(self.channel.clone(), &handle);
| ^^^^^^^ help: change this to: `handle`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
this expression creates a reference which is immediately dereferenced by the compiler:
cw-orch-daemon/src/senders/base_sender.rs#L194
error: this expression creates a reference which is immediately dereferenced by the compiler
--> cw-orch-daemon/src/senders/base_sender.rs:194:37
|
194 | Ok(Sender::new_with_options(&state, sender_options)?)
| ^^^^^^ help: change this to: `state`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `-D clippy::needless-borrow` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::needless_borrow)]`
|
question mark operator is useless here:
cw-orch-daemon/src/senders/base_sender.rs#L194
error: question mark operator is useless here
--> cw-orch-daemon/src/senders/base_sender.rs:194:9
|
194 | Ok(Sender::new_with_options(&state, sender_options)?)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing question mark and `Ok()`: `Sender::new_with_options(&state, sender_options)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark
= note: `-D clippy::needless-question-mark` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::needless_question_mark)]`
|
fields `chain`, `handle`, `deployment_id`, and `sender` are never read:
cw-orch-daemon/src/sync/builder.rs#L186
error: fields `chain`, `handle`, `deployment_id`, and `sender` are never read
--> cw-orch-daemon/src/sync/builder.rs:186:20
|
184 | pub struct DaemonBuilderBase<SenderGen: SenderTrait> {
| ----------------- fields in this struct
185 | // # Required
186 | pub(crate) chain: Option<ChainInfoOwned>,
| ^^^^^
187 | // # Optional
188 | pub(crate) handle: Option<tokio::runtime::Handle>,
| ^^^^^^
189 | pub(crate) deployment_id: Option<String>,
| ^^^^^^^^^^^^^
...
192 | pub(crate) sender: SenderGen,
| ^^^^^^
|
= note: `DaemonBuilderBase` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
= note: `-D dead-code` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(dead_code)]`
|
variable does not need to be mutable:
cw-orch-daemon/src/sync/builder.rs#L139
error: variable does not need to be mutable
--> cw-orch-daemon/src/sync/builder.rs:139:13
|
139 | let mut chain = self
| ----^^^^^
| |
| help: remove this `mut`
|
= note: `-D unused-mut` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_mut)]`
|
unused import: `bitcoin::secp256k1::All`:
cw-orch-daemon/src/sync/builder.rs#L7
error: unused import: `bitcoin::secp256k1::All`
--> cw-orch-daemon/src/sync/builder.rs:7:5
|
7 | use bitcoin::secp256k1::All;
| ^^^^^^^^^^^^^^^^^^^^^^^
|
unused imports: `SenderBuilder`, `Sender`:
cw-orch-daemon/src/sync/builder.rs#L3
error: unused imports: `SenderBuilder`, `Sender`
--> cw-orch-daemon/src/sync/builder.rs:3:28
|
3 | senders::base_sender::{Sender, SenderBuilder, SenderOptions},
| ^^^^^^ ^^^^^^^^^^^^^
|
unused import: `senders::base_sender::Sender`:
cw-orch-daemon/src/builder.rs#L13
error: unused import: `senders::base_sender::Sender`
--> cw-orch-daemon/src/builder.rs:13:33
|
13 | use super::{error::DaemonError, senders::base_sender::Sender, state::DaemonState};
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
unused import: `bitcoin::secp256k1::All`:
cw-orch-daemon/src/builder.rs#L11
error: unused import: `bitcoin::secp256k1::All`
--> cw-orch-daemon/src/builder.rs:11:5
|
11 | use bitcoin::secp256k1::All;
| ^^^^^^^^^^^^^^^^^^^^^^^
|
unused imports: `DaemonAsync`, `SenderBuilder`:
cw-orch-daemon/src/builder.rs#L4
error: unused imports: `DaemonAsync`, `SenderBuilder`
--> cw-orch-daemon/src/builder.rs:4:23
|
4 | base_sender::{SenderBuilder, SenderOptions},
| ^^^^^^^^^^^^^
...
7 | DaemonAsync, DaemonAsyncBase, DaemonBuilder, Wallet,
| ^^^^^^^^^^^
|
= note: `-D unused-imports` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_imports)]`
|
this expression creates a reference which is immediately dereferenced by the compiler:
cw-orch-daemon/src/live_mock.rs#L85
error: this expression creates a reference which is immediately dereferenced by the compiler
--> cw-orch-daemon/src/live_mock.rs:85:82
|
85 | let querier = CosmWasm::<Wallet>::new_sync(self.channel.clone(), &handle);
| ^^^^^^^ help: change this to: `handle`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
this expression creates a reference which is immediately dereferenced by the compiler:
cw-orch-daemon/src/senders/base_sender.rs#L194
error: this expression creates a reference which is immediately dereferenced by the compiler
--> cw-orch-daemon/src/senders/base_sender.rs:194:37
|
194 | Ok(Sender::new_with_options(&state, sender_options)?)
| ^^^^^^ help: change this to: `state`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `-D clippy::needless-borrow` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::needless_borrow)]`
|
question mark operator is useless here:
cw-orch-daemon/src/senders/base_sender.rs#L194
error: question mark operator is useless here
--> cw-orch-daemon/src/senders/base_sender.rs:194:9
|
194 | Ok(Sender::new_with_options(&state, sender_options)?)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing question mark and `Ok()`: `Sender::new_with_options(&state, sender_options)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark
= note: `-D clippy::needless-question-mark` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::needless_question_mark)]`
|
fields `chain`, `handle`, `deployment_id`, and `sender` are never read:
cw-orch-daemon/src/sync/builder.rs#L186
error: fields `chain`, `handle`, `deployment_id`, and `sender` are never read
--> cw-orch-daemon/src/sync/builder.rs:186:20
|
184 | pub struct DaemonBuilderBase<SenderGen: SenderTrait> {
| ----------------- fields in this struct
185 | // # Required
186 | pub(crate) chain: Option<ChainInfoOwned>,
| ^^^^^
187 | // # Optional
188 | pub(crate) handle: Option<tokio::runtime::Handle>,
| ^^^^^^
189 | pub(crate) deployment_id: Option<String>,
| ^^^^^^^^^^^^^
...
192 | pub(crate) sender: SenderGen,
| ^^^^^^
|
= note: `DaemonBuilderBase` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
= note: `-D dead-code` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(dead_code)]`
|
variable does not need to be mutable:
cw-orch-daemon/src/sync/builder.rs#L139
error: variable does not need to be mutable
--> cw-orch-daemon/src/sync/builder.rs:139:13
|
139 | let mut chain = self
| ----^^^^^
| |
| help: remove this `mut`
|
= note: `-D unused-mut` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_mut)]`
|
unused import: `bitcoin::secp256k1::All`:
cw-orch-daemon/src/sync/builder.rs#L7
error: unused import: `bitcoin::secp256k1::All`
--> cw-orch-daemon/src/sync/builder.rs:7:5
|
7 | use bitcoin::secp256k1::All;
| ^^^^^^^^^^^^^^^^^^^^^^^
|
unused imports: `SenderBuilder`, `Sender`:
cw-orch-daemon/src/sync/builder.rs#L3
error: unused imports: `SenderBuilder`, `Sender`
--> cw-orch-daemon/src/sync/builder.rs:3:28
|
3 | senders::base_sender::{Sender, SenderBuilder, SenderOptions},
| ^^^^^^ ^^^^^^^^^^^^^
|
unused import: `senders::base_sender::Sender`:
cw-orch-daemon/src/builder.rs#L13
error: unused import: `senders::base_sender::Sender`
--> cw-orch-daemon/src/builder.rs:13:33
|
13 | use super::{error::DaemonError, senders::base_sender::Sender, state::DaemonState};
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
unused import: `bitcoin::secp256k1::All`:
cw-orch-daemon/src/builder.rs#L11
error: unused import: `bitcoin::secp256k1::All`
--> cw-orch-daemon/src/builder.rs:11:5
|
11 | use bitcoin::secp256k1::All;
| ^^^^^^^^^^^^^^^^^^^^^^^
|
unused imports: `DaemonAsync`, `SenderBuilder`:
cw-orch-daemon/src/builder.rs#L4
error: unused imports: `DaemonAsync`, `SenderBuilder`
--> cw-orch-daemon/src/builder.rs:4:23
|
4 | base_sender::{SenderBuilder, SenderOptions},
| ^^^^^^^^^^^^^
...
7 | DaemonAsync, DaemonAsyncBase, DaemonBuilder, Wallet,
| ^^^^^^^^^^^
|
= note: `-D unused-imports` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_imports)]`
|
ubuntu / stable
Process completed with exit code 101.
|
ubuntu / beta
The job was canceled because "stable" failed.
|
ubuntu / beta
Process completed with exit code 101.
|
ubuntu / stable / coverage
Process completed with exit code 1.
|