ci: golangci-lint disable-all + explicit enable so errcheck stays off

`linters: enable: [...]` doesn't suppress the default lint set;
errcheck was still running and failing on legacy tx.Rollback() etc.
Flip to disable-all + explicit enable for a deterministic active set.
This commit is contained in:
Matthew Meszaros
2026-05-23 16:48:45 +00:00
parent d57febd1e3
commit 0d143dcfcc
+5 -8
View File
@@ -3,14 +3,11 @@ run:
modules-download-mode: readonly
linters:
# Curated set focused on real bugs, not stylistic churn. errcheck,
# unparam, prealloc, gosec, exportloopref are intentionally
# disabled — the legacy codebase has thousands of unchecked
# `tx.Rollback()` calls (which is idiomatic — Rollback after a
# successful Commit is a no-op), preallocation suggestions on hot
# paths the author chose not to follow, gosec rules that don't
# apply to our control-plane code, and exportloopref's check is
# already covered by govet's loopclosure in modern Go.
# disable-all + explicit enable list so we don't accidentally
# inherit golangci-lint's defaults (which include errcheck — the
# legacy codebase has thousands of unchecked tx.Rollback() calls
# that aren't worth fixing in this PR).
disable-all: true
enable:
- gosimple
- govet