crossplane / crossplane/upjet

Enable `nolintlint` linter

Open
#469 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.