Error: @augu/clippy-action failed to run

warning: wildcard matches only a single variant and will also match any future added variants --> gitarena-macros/src/route.rs:44:13 | 44 | _ => { /* ignored - actix web will error if the attribute is invalid */ } | ^ help: try: `NestedMeta::Lit(_)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#match_wildcard_for_single_variants = note: `-W clippy::match-wildcard-for-single-variants` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::match_wildcard_for_single_variants)]`

warning: this function has too many lines (110/100) --> gitarena-macros/src/route.rs:10:1 | 10 | pub(crate) fn route(args: TokenStream, input: TokenStream) -> TokenStream { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#too_many_lines = note: `-W clippy::too-many-lines` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::too_many_lines)]`

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let` --> gitarena-macros/src/ipc_packet.rs:73:5 | 73 | / match (category, packet_id) { 74 | | (Some(category), Some(packet_id)) => { 75 | | let uppercased_category = { ... | 107 | | } | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#single_match_else help: try | 73 ~ if let (Some(category), Some(packet_id)) = (category, packet_id) { 74 + let uppercased_category = { 75 + // https://stackoverflow.com/a/53570840 76 + let mut chars = category.chars(); 77 + 78 + match chars.next() { 79 + Some(c) => c.to_uppercase().collect::<String>() + chars.as_str(), 80 + None => String::new(), 81 + } 82 + }; 83 + 84 + let enum_identifier = Ident::new(uppercased_category.as_str(), Span::call_site()); 85 + 86 + TokenStream::from(quote! { 87 + use gitarena_macros::ipc; 88 + 89 + impl crate::ipc::PacketId for #identifier { 90 + #[inline] 91 + fn id(&self) -> u64 { 92 + crate::packets::PacketCategory::#enum_identifier as u64 + #packet_id 93 + } 94 + } 95 + }) 96 + } else { 97 + emit_call_site_error! { 98 + "#[ipc] requires both `packet` and `id` arguments"; 99 + help = "example: #[ipc(packet = \"git\", id = 1)]"; 100 + help = "this will result in packet id 1001 (category git = 1000 + id 1)"; 101 + } 102 + 103 + TokenStream::new() 104 + } |

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let` --> gitarena-macros/src/ipc_packet.rs:40:49 | 40 | ... packet_id = match value.base10_parse::<u64>() { | ___________________________________^ 41 | | ... Ok(id) => Some(id), 42 | | ... Err(_) => { 43 | | ... emit_error! { ... | 50 | | ... }; | |_______________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#single_match_else = note: `-W clippy::single-match-else` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::single_match_else)]` help: try | 40 ~ packet_id = if let Ok(id) = value.base10_parse::<u64>() { Some(id) } else { 41 + emit_error! { 42 + value.span(), 43 + "id argument could not be parsed into u64" 44 + } 45 + 46 + None 47 ~ }; |

warning: explicit `deref_mut` method call --> gitarena-macros/src/route.rs:110:27 | 110 | let pattern = boxed.deref_mut(); | ^^^^^^^^^^^^^^^^^ help: try: `&mut **boxed` | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#explicit_deref_methods = note: `-W clippy::explicit-deref-methods` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::explicit_deref_methods)]`

warning: wildcard matches only a single variant and will also match any future added variants --> gitarena-macros/src/route.rs:100:13 | 100 | _ => unimplemented!(), | ^ help: try: `FnArg::Receiver(_)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#match_wildcard_for_single_variants

warning: wildcard matches only a single variant and will also match any future added variants --> gitarena-macros/src/route.rs:44:13 | 44 | _ => { /* ignored - actix web will error if the attribute is invalid */ } | ^ help: try: `NestedMeta::Lit(_)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#match_wildcard_for_single_variants = note: `-W clippy::match-wildcard-for-single-variants` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::match_wildcard_for_single_variants)]`

warning: this function has too many lines (110/100) --> gitarena-macros/src/route.rs:10:1 | 10 | pub(crate) fn route(args: TokenStream, input: TokenStream) -> TokenStream { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#too_many_lines = note: `-W clippy::too-many-lines` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::too_many_lines)]`

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let` --> gitarena-macros/src/ipc_packet.rs:73:5 | 73 | / match (category, packet_id) { 74 | | (Some(category), Some(packet_id)) => { 75 | | let uppercased_category = { ... | 107 | | } | |_____^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.95.0/index.html#single_match_else help: try | 73 ~ if let (Some(category), Some(packet_id)) = (category, packet_id) { 74 + let uppercased_category = { 75 + // https://stackoverflow.com/a/53570840 76 + let mut chars = category.chars(); 77 + 78 + match chars.next() { 79 + Some(c) => c.to_uppercase().collect::<String>() + chars.as_str(), 80 + None => String::new(), 81 + } 82 + }; 83 + 84 + let enum_identifier = Ident::new(uppercased_category.as_str(), Span::call_site()); 85 + 86 + TokenStream::from(quote! { 87 + use gitarena_macros::ipc; 88 + 89 + impl crate::ipc::PacketId for #identifier { 90 + #[inline] 91 + fn id(&self) -> u64 { 92 + crate::packets::PacketCategory::#enum_identifier as u64 + #packet_id 93 + } 94 + } 95 + }) 96 + } else { 97 + emit_call_site_error! { 98 + "#[ipc] requires both `packet` and `id` arguments"; 99 + help = "example: #[ipc(packet = \"git\", id = 1)]"; 100 + help = "this will result in packet id 1001 (category git = 1000 + id 1)"; 101 + } 102 + 103 + TokenStream::new() 104 + } |