Skip to content

Commit 7869409

Browse files
committed
Make clippy:: a part of the RFC
1 parent 453428e commit 7869409

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

text/0000-clippy-uno.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,12 @@ In general clippy is intended to be used with a liberal sprinkling of `#[allow()
3535
disagree with Clippy's choices. This is a weaker philosophy than that behind rustc's lints, where usually flipping
3636
one is an indication of a very specialized situation.
3737

38+
## Lint attributes
3839

39-
Currently to do this well you often have to `#[cfg_attr(clippy, allow(lintname))]` which is somewhat tedious. Ideally
40-
the compiler will support something like `#[allow(clippy::lintname)]` which won't warn about nonexistant lints
41-
at all if there is no lint engine named `clippy`. This probably needs to be figured out before Clippy 1.0.
40+
Currently to allow/deny clippy lints you have to `#[cfg_attr(clippy, allow(lintname))]` which is somewhat tedious.
41+
42+
The compiler should support something like `#[allow(clippy::lintname)]` which won't attempt to warn about nonexistant lints
43+
at all when not running clippy.
4244

4345

4446
## Stability guarantees
@@ -259,6 +261,3 @@ both reflect and affect the general style of the community.
259261
Through the process of this RFC we hope to determine if there are lints which need
260262
to be uplifted, recategorized, or removed.
261263

262-
The question of how `#[allow(clippy::foo)]` might work can be solved in this RFC, but
263-
need not be. We have to make a decision on this before Clippy 1.0, however.
264-

0 commit comments

Comments
 (0)