Proposal: check for unused //nolint comments

[vearutop]

Sometimes the code needs a//nolint suppressor to disable check(s), but as the code evolves the //nolintmay become redundant.

For example such comment may become obsolete when the symbol is actually used:

// MyConst is something special.
// nolint:unused // To be used in subsequent PR.
const MyConst = "my-const"

I would like to check that //nolint comment was relevant to at least one linting issue and emit lint error otherwise.

[tpounds]

@vearutop There was pr #740 that was recently closed to implement similar functionality.

[tpounds]

cc @ashanbrown @jirfag

[jirfag]

I like this idea and it's not difficult to implement in nolint filter

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[ldez]

nolintlint handle this case now:

  nolintlint:
    # Enable to ensure that nolint directives are all used. Default is true.
    allow-unused: false