-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unable to silence lints #217
Comments
I should think about that. I guessed that the attributes before the function should act as test's attributes but maybe I was wrong. Maybe only a small subset like I'm little scared because when the user use the injected tests attribute I cannot know what attributes this test attribute will handle... I can handle all standard attributes as tests attributes and move the others to the function but I should also introduce a syntax that the user can use to force some attribute to be applied to the test instead the function. I'll com back here to draft the new syntax. |
This correctly triggers
clippy::unwrap_used
. However, placing#[allow(clippy::unwrap_used)]
on the function does not silence the lint. This is the case regardless of the position of the attribute.Here is the current expansion (without an
#[allow]
attribute).Perhaps
#[allow]
,#[warn]
,#[deny]
, and#[forbid]
should be special-cased such that they are copied onto the resulting function? I have no idea how feasible this is.The text was updated successfully, but these errors were encountered: