golangci-lint
golangci-lint copied to clipboard
Proposal: rename internal nolintlint linter to nolint
Is your feature request related to a problem? Please describe.
I'm concerned that the nolintlint
name is a bit confusing and that that nolint
might be a more obvious name for nolintlint
.
Describe the solution you'd like
Rename nolintlint
in configs and internal objects to nolint
. Continue to support nolintlint
as a deprecated config option.
Describe alternatives you've considered
Leave it alone or call the linter itself nolinter
while possibly still allowing the config options to be set under the keynolint
.
Additional context
Once we merge #1583 , we start allowing //nolint:nolintlint
to disable nolintlint
checks but I think these might look bitter as //nolint:nolint
. If we make this change, it would be nice to do so before people start adding //nolint:nolintlint
to their code. I don't believe //nolint:nolintlint
works right now.
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.