trillian icon indicating copy to clipboard operation
trillian copied to clipboard

Bump golangci-lint and remove //nolint comments in experimental/batchmap/cmd/verify/verify.go

Open AlCutter opened this issue 3 years ago • 0 comments

#2787 migrated Trillian to use klog. At the time, golangci-lint's staticcheck did not know that klog.Exit/Fatal would not return and so threw false +ve lint checks during presubmit which were suppressed with nolint comments.

A fix has been sent which enlightens the lint checker, so once that fix has made it into a golangci-lint release (v1.48.0 did not contain it) we should bump the version used in our presubmits and remove the //nolint comments in experimental/batchmap/cmd/verify/verify.go

AlCutter avatar Aug 18 '22 12:08 AlCutter