golangci-lint icon indicating copy to clipboard operation
golangci-lint copied to clipboard

Proposal: rename internal nolintlint linter to nolint

Open ashanbrown opened this issue 4 years ago • 1 comments

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.

ashanbrown avatar Dec 31 '20 16:12 ashanbrown

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.

stale[bot] avatar Jan 08 '22 23:01 stale[bot]