Skip to content

Commit

Permalink
Allow clippy::disallowed_names since it can be used for debugging
Browse files Browse the repository at this point in the history
https://rust-lang.github.io/rust-clippy/master/index.html#/disallowed_names

The current list is extremely short ("foo", "baz", "quux"), and thus
this really just prevents people from temporarily using `foo` to debug
code. In the spirit of improving productivity and reducing frustration,
it's easier to just allow this.

Bug: N/A
Test: TH
Change-Id: I50b65486ed2b58bfd2cf699ee3b43b936d25f2cb
  • Loading branch information
stephenhines committed Feb 29, 2024
1 parent 6c2cd90 commit 5f8dae5
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions rust/config/lints.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ var (
// It should be assumed that any warning lint will be promoted to a
// deny.
defaultClippyLints = []string{
// Let people hack in peace. ;)
"-A clippy::disallowed_names",
"-A clippy::type-complexity",
"-A clippy::unnecessary_fallible_conversions",
"-A clippy::unnecessary-wraps",
Expand Down

0 comments on commit 5f8dae5

Please sign in to comment.