Allow non_snake_case on impl #237

Closed
opened 2021-04-26 20:11:02 -05:00 by saiintbrisson · 2 comments
saiintbrisson commented 2021-04-26 20:11:02 -05:00 (Migrated from github.com)

I'm getting a warning about non-snake case function idents:
image

I noted that
github.com/bitflags/bitflags@c2ba431413/src/lib.rs (L505-L507)
and
github.com/bitflags/bitflags@c2ba431413/src/lib.rs (L594-L596)
do not allow non_snake_case while their trait definitions do.

Should rust-analyzer be suppressing this warning, or could we allow non_snake_case on the impls?

I'm getting a warning about non-snake case function idents: ![image](https://user-images.githubusercontent.com/29989290/116169588-68b9fc80-a6db-11eb-9d02-e51a5359a788.png) I noted that https://github.com/bitflags/bitflags/blob/c2ba43141307a97484518ad7728534855e1a51f2/src/lib.rs#L505-L507 and https://github.com/bitflags/bitflags/blob/c2ba43141307a97484518ad7728534855e1a51f2/src/lib.rs#L594-L596 do not allow `non_snake_case` while their trait definitions do. Should rust-analyzer be suppressing this warning, or could we allow `non_snake_case` on the impls?
saiintbrisson commented 2021-04-26 20:26:20 -05:00 (Migrated from github.com)

I've suggested suppressing the warning on rust-analyzer https://github.com/rust-analyzer/rust-analyzer/issues/8675.

I've suggested suppressing the warning on rust-analyzer https://github.com/rust-analyzer/rust-analyzer/issues/8675.
KodrAus commented 2021-07-18 19:39:44 -05:00 (Migrated from github.com)

We could also freely pepper #[allow()]s in more places where we generate those warnings ourselves.

We could also freely pepper `#[allow()]`s in more places where we generate those warnings ourselves.
Sign in to join this conversation.
No labels
bug
good first issue
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-bitflags/bitflags#237
No description provided.