Enable `nolintlint` linter
Open
enhancement
- Dominant language
- Go
- Stars
- 481
- Forks
- 131
- Avg merge
- 2d 1h
- Merged PRs (30d)
- 11
Description
There currently exists a pattern of `//nolint` directives disabling linters without saying why.
We should enable the `nolintlint` linter, that'll require explanatory comments for `//nolint` directives.
This helps the reader understand why a `//nolint` directive was appropriate.
As a bonus, it'll also automatically remove //nolint directives that don't do anything anymore.
Contributor guide
Assessment
This issue has not been assessed yet.