-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nolintlint: too many whitespaces removed after autofix #1580
Comments
I'm not completely sure it is possible to determine the user's intent in this case. We don't know whether the expected coding style is to use machine-readable directives or not (unless |
I think that something like that can be possible:
|
… required It isn't possible to determine what the whitespace convention is otherwise. Fixes golangci#1580.
… required It isn't possible to determine what the whitespace convention is otherwise. Fixes golangci#1580.
… required It isn't possible to determine what the whitespace convention is otherwise. Fixes golangci#1580.
Related to #1573
The fix removes all whitespaces, the choice to remove all the whitespaces, in this case, seems not right.
before the fix:
after the fix:
OK
before the fix:
after the fix:
KO
The text was updated successfully, but these errors were encountered: