You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We'd like to be able to say, in our configuration: you may //nolint these linters, but not those. Currently, there's no way to do this, as far as I know.
A few examples of why this could be useful:
linters which are necessary to ensure correctness/security, and which should not be bypassed as easily as stylistic linters
Ideally, one could enable this behavior for arbitrary files, regexp-matches, etc., just like excludes, but simply opting in/out on a per-linter basis is probably good enough for us.
The text was updated successfully, but these errors were encountered:
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.
We solved such problem by using a custom config in CI, just only those required linters are enabled and run golangci-lint with that config for all repos. The project owners can have their own config with any linters they want, but the code must pass the with the CI's config
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.
We'd like to be able to say, in our configuration: you may
//nolint
these linters, but not those. Currently, there's no way to do this, as far as I know.A few examples of why this could be useful:
//nolint
directives themselves (in the style of Add nolintlint linter #740 or Proposal: check for unused //nolint comments #832)Ideally, one could enable this behavior for arbitrary files, regexp-matches, etc., just like excludes, but simply opting in/out on a per-linter basis is probably good enough for us.
The text was updated successfully, but these errors were encountered: