Critical vulnerabilities were found, marking check as failed

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/audit-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/

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions-rs/audit-check@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

2 vulnerabilities found!

3 warnings found!

Unknown warning kind unsound found, please, file a bug

warning: this `impl` can be derived --> src/state/mod.rs:33:1 | 33 | / impl Default for GameState { 34 | | fn default() -> Self { 35 | | GameState::None 36 | | } 37 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it... | 4 + #[derive(Default)] 5 | pub enum GameState { | help: ...and mark the default variant | 5 ~ #[default] 6 ~ None, |

warning: the following explicit lifetimes could be elided: 'a --> src/error/mod.rs:25:22 | 25 | async fn reply_error<'a>(ctx: Context<'a>, error: impl Display) -> anyhow::Result<()> { | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/command/start.rs:18:30 | 18 | GameState::transition_to(&game_state, GameState::Discussion)?; | ^^^^^^^^^^^ help: change this to: `game_state` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/command/hold_fetish.rs:13:30 | 13 | GameState::transition_to(&game_state, GameState::Preparing)?; | ^^^^^^^^^^^ help: change this to: `game_state` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default

warning: variants `Voting` and `Evening` are never constructed --> src/state/mod.rs:8:5 | 4 | pub enum GameState { | --------- variants in this enum ... 8 | Voting, | ^^^^^^ 9 | Evening, | ^^^^^^^ | = note: `GameState` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis

warning: fields `name` and `is_alive` are never read --> src/model/attender.rs:8:9 | 6 | pub struct Attender { | -------- fields in this struct 7 | pub id: serenity::UserId, 8 | pub name: String, | ^^^^ 9 | pub fetish: String, 10 | pub is_alive: bool, | ^^^^^^^^ | = note: `Attender` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default

warning: this `impl` can be derived --> src/state/mod.rs:33:1 | 33 | / impl Default for GameState { 34 | | fn default() -> Self { 35 | | GameState::None 36 | | } 37 | | } | |_^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls = note: `#[warn(clippy::derivable_impls)]` on by default = help: remove the manual implementation... help: ...and instead derive it... | 4 + #[derive(Default)] 5 | pub enum GameState { | help: ...and mark the default variant | 5 ~ #[default] 6 ~ None, |

warning: the following explicit lifetimes could be elided: 'a --> src/error/mod.rs:25:22 | 25 | async fn reply_error<'a>(ctx: Context<'a>, error: impl Display) -> anyhow::Result<()> { | ^^ ^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes = note: `#[warn(clippy::needless_lifetimes)]` on by default

warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/command/start.rs:18:30 | 18 | GameState::transition_to(&game_state, GameState::Discussion)?; | ^^^^^^^^^^^ help: change this to: `game_state` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/command/hold_fetish.rs:13:30 | 13 | GameState::transition_to(&game_state, GameState::Preparing)?; | ^^^^^^^^^^^ help: change this to: `game_state` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default

warning: variants `Voting` and `Evening` are never constructed --> src/state/mod.rs:8:5 | 4 | pub enum GameState { | --------- variants in this enum ... 8 | Voting, | ^^^^^^ 9 | Evening, | ^^^^^^^ | = note: `GameState` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis