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
Describe the bug
We would like to have no exceptions even for test files on dot-imports. Unfortunately, the linter rule has a hard-coded check disabled.
To Reproduce
Have a _test.go file with dot imports and try using the linter.
Expected behavior
Linter should detect those.
Additional context
We might have it as a config option set by default to ignore test files or just remove the hardcoded check and let the users re-configure their linters to ignore tests separately.
I can provide a PR if you agree.
The text was updated successfully, but these errors were encountered:
Hi @denisvmedia thanks for creating the ticket.
I think we can just remove the hardcoded check and let users use the new exclude argument available on all rules.
Describe the bug
We would like to have no exceptions even for test files on dot-imports. Unfortunately, the linter rule has a hard-coded check disabled.
To Reproduce
Have a _test.go file with dot imports and try using the linter.
Expected behavior
Linter should detect those.
Additional context
We might have it as a config option set by default to ignore test files or just remove the hardcoded check and let the users re-configure their linters to ignore tests separately.
I can provide a PR if you agree.
The text was updated successfully, but these errors were encountered: