Skip to content

Fix missing command #143

Fix missing command

Fix missing command #143

Triggered via pull request February 9, 2024 10:15
@juchiastjuchiast
synchronize #54
fix
Status Success
Total duration 6m 46s
Artifacts

basic.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

69 warnings
you seem to use `.enumerate()` and immediately discard the index: crates/cmds-solana/src/wormhole/verify_signatures.rs#L74
warning: you seem to use `.enumerate()` and immediately discard the index --> crates/cmds-solana/src/wormhole/verify_signatures.rs:74:31 | 74 | for (_tx_index, chunk) in signature_items.chunks(7).enumerate() { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_enumerate_index = note: `#[warn(clippy::unused_enumerate_index)]` on by default help: remove the `.enumerate()` call | 74 | for chunk in signature_items.chunks(7) { | ~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~
a `const` item should never be interior mutable: crates/flow-server/src/middleware/auth.rs#L73
warning: a `const` item should never be interior mutable --> crates/flow-server/src/middleware/auth.rs:73:1 | 73 | pub const X_API_KEY: HeaderName = HeaderName::from_static("x-api-key"); | -----^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | make this a static item (maybe with lazy_static) | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#declare_interior_mutable_const = note: `#[warn(clippy::declare_interior_mutable_const)]` on by default
a `const` item should never be interior mutable: crates/flow-server/src/middleware/auth.rs#L73
warning: a `const` item should never be interior mutable --> crates/flow-server/src/middleware/auth.rs:73:1 | 73 | pub const X_API_KEY: HeaderName = HeaderName::from_static("x-api-key"); | -----^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | make this a static item (maybe with lazy_static) | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#declare_interior_mutable_const = note: `#[warn(clippy::declare_interior_mutable_const)]` on by default
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/update.rs#L136
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/update.rs:136:28 | 136 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 137 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/transfer.rs#L190
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/transfer.rs:190:28 | 190 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 191 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/burn.rs#L133
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/burn.rs:133:28 | 133 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 134 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
you seem to use `.enumerate()` and immediately discard the index: crates/cmds-solana/src/wormhole/verify_signatures.rs#L74
warning: you seem to use `.enumerate()` and immediately discard the index --> crates/cmds-solana/src/wormhole/verify_signatures.rs:74:31 | 74 | for (_tx_index, chunk) in signature_items.chunks(7).enumerate() { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unused_enumerate_index = note: `#[warn(clippy::unused_enumerate_index)]` on by default help: remove the `.enumerate()` call | 74 | for chunk in signature_items.chunks(7) { | ~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/update.rs#L136
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/update.rs:136:28 | 136 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 137 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/transfer.rs#L190
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/transfer.rs:190:28 | 190 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 191 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
useless conversion to the same type: `u32`: crates/cmds-solana/src/compression/burn.rs#L133
warning: useless conversion to the same type: `u32` --> crates/cmds-solana/src/compression/burn.rs:133:28 | 133 | Some(asset) => (asset.result.node_index as u32 - 2 * asset.result.proof.len() as u32) | ____________________________^ 134 | | .try_into() | |___________________________^ | = help: consider removing `.try_into()` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`: crates/cmds-std/src/wait_cmd.rs#L30
warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let` --> crates/cmds-std/src/wait_cmd.rs:30:5 | 30 | / match input.duration_ms { 31 | | Some(duration) => { 32 | | time::sleep(Duration::from_millis(duration)).await; 33 | | } 34 | | None => {} 35 | | }; | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match = note: `#[warn(clippy::single_match)]` on by default help: try | 30 ~ if let Some(duration) = input.duration_ms { 31 + time::sleep(Duration::from_millis(duration)).await; 32 ~ }; |
module has the same name as its containing module: crates/cmds-std/src/supabase/mod.rs#L1
warning: module has the same name as its containing module --> crates/cmds-std/src/supabase/mod.rs:1:1 | 1 | pub mod supabase; | ^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#module_inception = note: `#[warn(clippy::module_inception)]` on by default
redundant closure: crates/db/src/connection/conn_impl.rs#L815
warning: redundant closure --> crates/db/src/connection/conn_impl.rs:815:62 | 815 | let signatures = signatures.map(|arr| arr.iter().map(|p| Json(p)).collect::<Vec<_>>()); | ^^^^^^^^^^^ help: replace the closure with the function itself: `Json` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure = note: `#[warn(clippy::redundant_closure)]` on by default
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L360
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:360:17 | 360 | let res = self.save_signature(&id, &signature).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.save_signature(&id, &signature).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L335
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:335:17 | 335 | let res = self.set_node_finish(&id, &node_id, &times, &time).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_node_finish(&id, &node_id, &times, &time).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L330
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:330:17 | 330 | let res = self.push_node_error(&id, &node_id, &times, &error).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.push_node_error(&id, &node_id, &times, &error).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L322
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:322:17 | 322 | / let res = self 323 | | .save_node_output(&id, &node_id, &times, &output) 324 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 322 ~ self 323 + .save_node_output(&id, &node_id, &times, &output) 324 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L315
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:315:17 | 315 | / let res = self 316 | | .new_node_run(&id, &node_id, &times, &time, &input) 317 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 315 ~ self 316 + .new_node_run(&id, &node_id, &times, &time, &input) 317 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L310
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:310:17 | 310 | let res = self.set_run_result(&id, &time, &not_run, &output).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_run_result(&id, &time, &not_run, &output).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L302
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:302:17 | 302 | / let res = self 303 | | .push_flow_log(&id, &index, &time, &level, &module, &content) 304 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 302 ~ self 303 + .push_flow_log(&id, &index, &time, &level, &module, &content) 304 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L296
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:296:17 | 296 | let res = self.push_flow_error(&id, &error).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.push_flow_error(&id, &error).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L291
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:291:17 | 291 | let res = self.set_start_time(&id, &time).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_start_time(&id, &time).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
redundant closure: crates/db/src/connection/conn_impl.rs#L815
warning: redundant closure --> crates/db/src/connection/conn_impl.rs:815:62 | 815 | let signatures = signatures.map(|arr| arr.iter().map(|p| Json(p)).collect::<Vec<_>>()); | ^^^^^^^^^^^ help: replace the closure with the function itself: `Json` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure = note: `#[warn(clippy::redundant_closure)]` on by default
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L256
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:256:17 | 256 | let res = self.share_flow_run(id, user).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.share_flow_run(id, user).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value = note: `#[warn(clippy::let_unit_value)]` on by default
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L360
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:360:17 | 360 | let res = self.save_signature(&id, &signature).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.save_signature(&id, &signature).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L335
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:335:17 | 335 | let res = self.set_node_finish(&id, &node_id, &times, &time).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_node_finish(&id, &node_id, &times, &time).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L330
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:330:17 | 330 | let res = self.push_node_error(&id, &node_id, &times, &error).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.push_node_error(&id, &node_id, &times, &error).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L322
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:322:17 | 322 | / let res = self 323 | | .save_node_output(&id, &node_id, &times, &output) 324 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 322 ~ self 323 + .save_node_output(&id, &node_id, &times, &output) 324 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L315
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:315:17 | 315 | / let res = self 316 | | .new_node_run(&id, &node_id, &times, &time, &input) 317 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 315 ~ self 316 + .new_node_run(&id, &node_id, &times, &time, &input) 317 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L310
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:310:17 | 310 | let res = self.set_run_result(&id, &time, &not_run, &output).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_run_result(&id, &time, &not_run, &output).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L302
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:302:17 | 302 | / let res = self 303 | | .push_flow_log(&id, &index, &time, &level, &module, &content) 304 | | .await?; | |____________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value help: omit the `let` binding | 302 ~ self 303 + .push_flow_log(&id, &index, &time, &level, &module, &content) 304 + .await?; |
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L296
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:296:17 | 296 | let res = self.push_flow_error(&id, &error).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.push_flow_error(&id, &error).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L291
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:291:17 | 291 | let res = self.set_start_time(&id, &time).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.set_start_time(&id, &time).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value
this let-binding has unit value: crates/db/src/connection/proxied_user_conn.rs#L256
warning: this let-binding has unit value --> crates/db/src/connection/proxied_user_conn.rs:256:17 | 256 | let res = self.share_flow_run(id, user).await?; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: omit the `let` binding: `self.share_flow_run(id, user).await?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value = note: `#[warn(clippy::let_unit_value)]` on by default
use of deprecated method `indexmap::map::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior.: crates/cmds-std/src/json_extract.rs#L55
warning: use of deprecated method `indexmap::map::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior. --> crates/cmds-std/src/json_extract.rs:55:31 | 55 | let trimmed = outputs.remove("trimmed_json").unwrap(); | ^^^^^^
use of deprecated method `indexmap::map::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior.: crates/cmds-std/src/json_extract.rs#L54
warning: use of deprecated method `indexmap::map::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior. --> crates/cmds-std/src/json_extract.rs:54:29 | 54 | let value = outputs.remove("value").unwrap(); | ^^^^^^ | = note: `#[warn(deprecated)]` on by default
you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`: crates/cmds-std/src/wait_cmd.rs#L30
warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let` --> crates/cmds-std/src/wait_cmd.rs:30:5 | 30 | / match input.duration_ms { 31 | | Some(duration) => { 32 | | time::sleep(Duration::from_millis(duration)).await; 33 | | } 34 | | None => {} 35 | | }; | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match = note: `#[warn(clippy::single_match)]` on by default help: try | 30 ~ if let Some(duration) = input.duration_ms { 31 + time::sleep(Duration::from_millis(duration)).await; 32 ~ }; |
module has the same name as its containing module: crates/cmds-std/src/supabase/mod.rs#L1
warning: module has the same name as its containing module --> crates/cmds-std/src/supabase/mod.rs:1:1 | 1 | pub mod supabase; | ^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#module_inception = note: `#[warn(clippy::module_inception)]` on by default
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L447
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:447:9 | 447 | / match self.fx0 { 448 | | Fx0::Wood => { 449 | | self.wood_variation = WoodVariation::choose(rng); 450 | | } 451 | | _ => {} 452 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 447 ~ if self.fx0 == Fx0::Wood { 448 + self.wood_variation = WoodVariation::choose(rng); 449 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L437
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:437:9 | 437 | / match self.fx0 { 438 | | Fx0::Marble => { 439 | | self.marble_variation = MarbleVariation::choose(rng); 440 | | } 441 | | _ => {} 442 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 437 ~ if self.fx0 == Fx0::Marble { 438 + self.marble_variation = MarbleVariation::choose(rng); 439 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L427
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:427:9 | 427 | / match self.fx0 { 428 | | Fx0::No => { 429 | | self.body_material_variation = BodyMaterialVariations::choose(rng); 430 | | } 431 | | _ => {} 432 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 427 ~ if self.fx0 == Fx0::No { 428 + self.body_material_variation = BodyMaterialVariations::choose(rng); 429 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L380
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:380:9 | 380 | / match self.fx2 { 381 | | Fx2::Underwater => { 382 | | let jellyfish = FxJellyfish::choose(rng); 383 | | self.fx_jellifish = jellyfish; ... | 395 | | _ => {} 396 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match = note: `#[warn(clippy::single_match)]` on by default help: try | 380 ~ if self.fx2 == Fx2::Underwater { 381 + let jellyfish = FxJellyfish::choose(rng); 382 + self.fx_jellifish = jellyfish; 383 + 384 + self.underwater_fog_amount = rng.gen::<f64>() * 30.0; 385 + self.background_underwater_color_hue = 38.8; 386 + 387 + let env_light = if self.fx0 == Fx0::Hologram { 388 + EnvLight::UnderwaterHologram 389 + } else { 390 + EnvLight::Underwater 391 + }; 392 + self.env_light = env_light; 393 + } |
this returns a `Result<_, ()>`: crates/utils/src/address_book.rs#L64
warning: this returns a `Result<_, ()>` --> crates/utils/src/address_book.rs:64:5 | 64 | / pub fn try_start_with_context<A, F>( 65 | | &mut self, 66 | | id: A::ID, 67 | | make_actor: F, 68 | | rt: ArbiterHandle, 69 | | ) -> Result<actix::Addr<A>, ()> | |___________________________________^ | = help: use a custom `Error` type instead = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_unit_err = note: `#[warn(clippy::result_unit_err)]` on by default
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L447
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:447:9 | 447 | / match self.fx0 { 448 | | Fx0::Wood => { 449 | | self.wood_variation = WoodVariation::choose(rng); 450 | | } 451 | | _ => {} 452 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 447 ~ if self.fx0 == Fx0::Wood { 448 + self.wood_variation = WoodVariation::choose(rng); 449 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L437
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:437:9 | 437 | / match self.fx0 { 438 | | Fx0::Marble => { 439 | | self.marble_variation = MarbleVariation::choose(rng); 440 | | } 441 | | _ => {} 442 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 437 ~ if self.fx0 == Fx0::Marble { 438 + self.marble_variation = MarbleVariation::choose(rng); 439 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L427
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:427:9 | 427 | / match self.fx0 { 428 | | Fx0::No => { 429 | | self.body_material_variation = BodyMaterialVariations::choose(rng); 430 | | } 431 | | _ => {} 432 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match help: try | 427 ~ if self.fx0 == Fx0::No { 428 + self.body_material_variation = BodyMaterialVariations::choose(rng); 429 + } |
you seem to be trying to use `match` for an equality check. Consider using `if`: crates/pdg-common/src/nft_metadata/generate.rs#L380
warning: you seem to be trying to use `match` for an equality check. Consider using `if` --> crates/pdg-common/src/nft_metadata/generate.rs:380:9 | 380 | / match self.fx2 { 381 | | Fx2::Underwater => { 382 | | let jellyfish = FxJellyfish::choose(rng); 383 | | self.fx_jellifish = jellyfish; ... | 395 | | _ => {} 396 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match = note: `#[warn(clippy::single_match)]` on by default help: try | 380 ~ if self.fx2 == Fx2::Underwater { 381 + let jellyfish = FxJellyfish::choose(rng); 382 + self.fx_jellifish = jellyfish; 383 + 384 + self.underwater_fog_amount = rng.gen::<f64>() * 30.0; 385 + self.background_underwater_color_hue = 38.8; 386 + 387 + let env_light = if self.fx0 == Fx0::Hologram { 388 + EnvLight::UnderwaterHologram 389 + } else { 390 + EnvLight::Underwater 391 + }; 392 + self.env_light = env_light; 393 + } |
use of deprecated method `indexmap::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior.: lib/flow-value/src/lib.rs#L884
warning: use of deprecated method `indexmap::IndexMap::<K, V, S>::remove`: `remove` disrupts the map order -- use `swap_remove` or `shift_remove` for explicit behavior. --> lib/flow-value/src/lib.rs:884:15 | 884 | v.remove("value").unwrap(), | ^^^^^^ | = note: `#[warn(deprecated)]` on by default
this returns a `Result<_, ()>`: crates/utils/src/address_book.rs#L64
warning: this returns a `Result<_, ()>` --> crates/utils/src/address_book.rs:64:5 | 64 | / pub fn try_start_with_context<A, F>( 65 | | &mut self, 66 | | id: A::ID, 67 | | make_actor: F, 68 | | rt: ArbiterHandle, 69 | | ) -> Result<actix::Addr<A>, ()> | |___________________________________^ | = help: use a custom `Error` type instead = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#result_unit_err = note: `#[warn(clippy::result_unit_err)]` on by default
Clippy lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions-rs/toolchain@v1, actions/checkout@v3, Swatinem/rust-cache@v1, 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 lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, Swatinem/rust-cache@v1, 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/
Clippy lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Clippy lint
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions-rs/toolchain@v1, actions/checkout@v3, Swatinem/rust-cache@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Unit test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, Swatinem/rust-cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Unit test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Unit test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/