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
The result of which (among other errors in my file)...
main.go:13:1: no space between comment delimiter and comment text
Expected behavior
I expected the line //lint:ignore mnd to be ignored (I actually noticed this issue indirectly by revive via golangci-lint but was able to reproduce with revive directly).
Logs
Note
Interestingly you'll see in the screenshot another comment line that does get ignored even though it shouldn't.
Desktop (please complete the following information):
OS: macOS Sonoma 14.4.1
Version of Go: go version go1.22.4 darwin/arm64
Additional context
Add any other context about the problem here.
I also tried modifying the config to use arguments = ["lint:"] rather than arguments = ["lint:ignore"] but that also didn't help.
The text was updated successfully, but these errors were encountered:
Hi @Integralist, thanks for reporting the problem (and sorry for the late response)
I'm not able to reproduce the bug with the later version of revive. Could you please check if you still reproduce it?
Describe the bug
I have defined a comment-spacings exclusion which isn't being respected
To Reproduce
Steps to reproduce the behavior:
go install github.com/mgechev/revive@latest
revive -config ~/revive.toml main.go
The result of which (among other errors in my file)...
Expected behavior
I expected the line
//lint:ignore mnd
to be ignored (I actually noticed this issue indirectly by revive via golangci-lint but was able to reproduce with revive directly).Logs
Note
Interestingly you'll see in the screenshot another comment line that does get ignored even though it shouldn't.
Desktop (please complete the following information):
go version go1.22.4 darwin/arm64
Additional context
Add any other context about the problem here.
I also tried modifying the config to use
arguments = ["lint:"]
rather thanarguments = ["lint:ignore"]
but that also didn't help.The text was updated successfully, but these errors were encountered: