Skip to content

fix link

fix link #709

Triggered via push October 10, 2023 13:59
Status Success
Total duration 21s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention
Fit to window
Zoom out
Zoom in

Annotations

18 errors and 1 warning
the borrowed expression implements the required traits: catalyst-toolbox/src/stats/live/harvester.rs#L50
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/stats/live/harvester.rs:50:18 | 50 | .get(&format!("{}/v0/fragment/logs", self.endpoint)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/v0/fragment/logs", self.endpoint)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
this OR pattern can be rewritten using a range: catalyst-toolbox/src/snapshot.rs#L183
error: this OR pattern can be rewritten using a range --> catalyst-toolbox/src/snapshot.rs:183:9 | 183 | 0x0 | 0x1 | 0x2 | 0x3 | 0x4 | 0x5 | 0x6 | 0x7 => AddrType::Shelley, | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `0..=7` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_patterns = note: `-D clippy::manual-range-patterns` implied by `-D warnings`
casting integer literal to `u64` is unnecessary: catalyst-toolbox/src/snapshot.rs#L80
error: casting integer literal to `u64` is unnecessary --> catalyst-toolbox/src/snapshot.rs:80:37 | 80 | ... value / 1_000_000 as u64 | ^^^^^^^^^^^^^^^^ help: try: `1_000_000_u64` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_cast = note: `-D clippy::unnecessary-cast` implied by `-D warnings`
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: catalyst-toolbox/src/rewards/voters.rs#L88
error: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> catalyst-toolbox/src/rewards/voters.rs:88:16 | 88 | .chain(vote_count.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `vote_count` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/iter/traits/iterator.rs:523:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: catalyst-toolbox/src/rewards/veterans.rs#L128
error: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> catalyst-toolbox/src/rewards/veterans.rs:128:14 | 128 | .zip(rewards_eligible_rankings.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `rewards_eligible_rankings` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `-D clippy::useless-conversion` implied by `-D warnings`
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L582
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:582:10 | 582 | ) -> Result<ReplayedFragment, (Error, Fragment)> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the `Err`-variant is at least 344 bytes | = help: try reducing the size of `(recovery::tally::Error, chain_impl_mockchain::fragment::Fragment)`, for example by boxing large elements or replacing it with `Box<(recovery::tally::Error, chain_impl_mockchain::fragment::Fragment)>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the borrowed expression implements the required traits: catalyst-toolbox/src/recovery/tally.rs#L548
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/recovery/tally.rs:548:25 | 548 | &mut rand::thread_rng(), | ^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `rand::thread_rng()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L529
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:529:63 | 42 | LedgerError(#[from] chain_impl_mockchain::ledger::Error), | -------------------------------------------------------- the largest variant contains at least 264 bytes ... 529 | fn replay_tx(&mut self, tx: TransactionSlice<NoExtra>) -> Result<Fragment, Error> { | ^^^^^^^^^^^^^^^^^^^^^^^ | = help: try reducing the size of `recovery::tally::Error`, for example by boxing large elements or replacing it with `Box<recovery::tally::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L505
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:505:70 | 42 | LedgerError(#[from] chain_impl_mockchain::ledger::Error), | -------------------------------------------------------- the largest variant contains at least 264 bytes ... 505 | fn replay_votecast(&mut self, tx: TransactionSlice<VoteCast>) -> Result<Fragment, Error> { | ^^^^^^^^^^^^^^^^^^^^^^^ | = help: try reducing the size of `recovery::tally::Error`, for example by boxing large elements or replacing it with `Box<recovery::tally::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L438
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:438:39 | 42 | LedgerError(#[from] chain_impl_mockchain::ledger::Error), | -------------------------------------------------------- the largest variant contains at least 264 bytes ... 438 | fn from_block0(block0: &Block) -> Result<(Self, Block), Error> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: try reducing the size of `recovery::tally::Error`, for example by boxing large elements or replacing it with `Box<recovery::tally::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L353
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:353:6 | 42 | LedgerError(#[from] chain_impl_mockchain::ledger::Error), | -------------------------------------------------------- the largest variant contains at least 264 bytes ... 353 | ) -> Result<(Ledger, Vec<Fragment>), Error> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: try reducing the size of `recovery::tally::Error`, for example by boxing large elements or replacing it with `Box<recovery::tally::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/tally.rs#L248
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/tally.rs:248:73 | 42 | LedgerError(#[from] chain_impl_mockchain::ledger::Error), | -------------------------------------------------------- the largest variant contains at least 264 bytes ... 248 | pub fn new(block0: Block, range_check: Range<u32>, fragments: I) -> Result<Self, Error> { | ^^^^^^^^^^^^^^^^^^^ | = help: try reducing the size of `recovery::tally::Error`, for example by boxing large elements or replacing it with `Box<recovery::tally::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
the borrowed expression implements the required traits: catalyst-toolbox/src/recovery/replay.rs#L56
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/recovery/replay.rs:56:47 | 56 | .format_json(serde_json::to_value(&voteplan_status)?)?; | ^^^^^^^^^^^^^^^^ help: change this to: `voteplan_status` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
the `Err`-variant returned from this function is very large: catalyst-toolbox/src/recovery/replay.rs#L39
error: the `Err`-variant returned from this function is very large --> catalyst-toolbox/src/recovery/replay.rs:39:26 | 39 | pub fn exec(self) -> Result<(), Error> { | ^^^^^^^^^^^^^^^^^ ... 72 | Recovery(#[from] crate::recovery::tally::Error), | ----------------------------------------------- the largest variant contains at least 264 bytes | = help: try reducing the size of `recovery::replay::Error`, for example by boxing large elements or replacing it with `Box<recovery::replay::Error>` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err = note: `-D clippy::result-large-err` implied by `-D warnings`
the borrowed expression implements the required traits: catalyst-toolbox/src/kedqr/payload.rs#L34
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/kedqr/payload.rs:34:33 | 34 | let key = decrypt(password, &encrypted_bytes)?; | ^^^^^^^^^^^^^^^^ help: change this to: `encrypted_bytes` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
the borrowed expression implements the required traits: catalyst-toolbox/src/kedqr/img.rs#L50
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/kedqr/img.rs:50:57 | 50 | let inner = QrCode::with_error_correction_level(&enc_hex, EcLevel::H).unwrap(); | ^^^^^^^^ help: change this to: `enc_hex` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
unnecessary closure used with `bool::then`: catalyst-toolbox/src/ideascale/mod.rs#L130
error: unnecessary closure used with `bool::then` --> catalyst-toolbox/src/ideascale/mod.rs:130:37 | 130 | challenge_type: funnels | _____________________________________^ 131 | | .get(&c.funnel_id) 132 | | .unwrap_or_else(|| panic!("A funnel with id {} wasn't found", c.funnel_id)) 133 | | .is_community() 134 | | .then(|| "community-choice") | |__________________________--------------------------^ | | | help: use `then_some(..)` instead: `then_some("community-choice")` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations = note: `-D clippy::unnecessary-lazy-evaluations` implied by `-D warnings`
the borrowed expression implements the required traits: catalyst-toolbox/src/ideascale/models/de.rs#L183
error: the borrowed expression implements the required traits --> catalyst-toolbox/src/ideascale/models/de.rs:183:30 | 183 | D::Error::custom(&format!( | ______________________________^ 184 | | "Unable to read malformed value: '{}'", 185 | | rewards_str 186 | | )) | |_____________^ | = 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: change this to | 183 ~ D::Error::custom(format!( 184 + "Unable to read malformed value: '{}'", 185 + rewards_str 186 ~ )) |
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/