-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Closed
Labels
area: nolintRelated to nolint directive and nolintlintRelated to nolint directive and nolintlintenhancementNew feature or improvementNew feature or improvement
Description
We'd like to be able to say, in our configuration: you may //nolint these linters, but not those. Currently, there's no way to do this, as far as I know.
A few examples of why this could be useful:
- linters which are necessary to ensure correctness/security, and which should not be bypassed as easily as stylistic linters
- customizing the rules for
//nolintdirectives themselves (in the style of Add nolintlint linter #740 or Proposal: check for unused //nolint comments #832)
Ideally, one could enable this behavior for arbitrary files, regexp-matches, etc., just like excludes, but simply opting in/out on a per-linter basis is probably good enough for us.
Metadata
Metadata
Assignees
Labels
area: nolintRelated to nolint directive and nolintlintRelated to nolint directive and nolintlintenhancementNew feature or improvementNew feature or improvement