cmd/go: go vet complains about redundant and/or #46046
Labels
FrozenDueToAge
NeedsDecision
Feedback is required from experts, contributors, and/or the community before a change can be made.
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?Note: I removed GOPRIVATE line
go env
OutputWhat did you do?
Minimal reproducible code:
run
go vet
against it complains:What did you expect to see?
We use this style because this allows us to generate better diff if we need to add/remove conditions in the future, it's in the same spirit of always allow trailing comma on arg list, even for the last arg. So we think we should allow this style through
go vet
.We also have some old, similar code that
go vet
does not complain about. I cannot create a minimal reproducible example for it that go vet does not complains, but the pseudo code is something like this (this is the implementation of our health check handler):What did you see instead?
The text was updated successfully, but these errors were encountered: