Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Expand #[expect] attributes into #[allow] #4409

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

samueltardieu
Copy link

An #[expect] outer attribute must not be kept in the #[automatically_derived] code, as the expanded code will probably not meet the expectations. Transforming it into the equivalent #[allow] attribute is the safest way to ensure that expected attributes will not trigger a warning.

Noticed through a bug report entered at the Clippy repository at rust-lang/rust-clippy#14008.

An `#[expect]` outer attribute must not be kept in the
`#[automatically_derived]` code, as the expanded code will probably not
meet the expectations. Transforming it into the equivalent `#[allow]`
attribute is the safest way to ensure that expected attributes will not
trigger a warning.

Noticed through a bug report entered at the Clippy repository at
<rust-lang/rust-clippy#14008>.
@samueltardieu
Copy link
Author

Is there anything missing on this PR to make it noticed?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant