Template
mathias
607c5fec3a
Estate had 26 Go repos, 3 with a lint config, all three different; the other 23 ran golangci-lint v2 defaults, which enable NO formatter -- the hole that let gofmt drift rot undetected until it was found by hand (dma-cli#12). Canonical lives at infra ci-templates/golangci.yml and is distributed by the infra#309 pattern: byte-identical copies, checksum drift gate in infra. Do not edit this file here -- edit the canonical and run `task ci:sync`. Verified locally before pushing, not pushed-and-hoped: golangci-lint v2.11.4 run against a Go 1.26.1 toolchain (matching CI) reports 0 issues in this repo with this config.
53 lines
1.8 KiB
YAML
53 lines
1.8 KiB
YAML
# Canonical golangci-lint configuration for the estate's Go repos (infra#309
|
|
# pattern: one canonical file here, a byte-identical copy in each adopter,
|
|
# and the drift gate in THIS repo comparing sha256).
|
|
#
|
|
# Why this exists: 26 Go repos, 3 had a lint config, and all three were
|
|
# different. The 23 without one ran golangci-lint's v2 default set, which
|
|
# does NOT include a formatter -- so gofmt/gofumpt drift went uncaught until
|
|
# it was found by hand (dma-cli#12: two files drifted, nothing flagged them).
|
|
#
|
|
# The two configs this replaces (cobalt-dingo, coo-agent) were near-identical
|
|
# copies of each other AND both invalid under v2: they carried v1-style
|
|
# `linters-settings` and `issues.exclude-rules` keys, which `golangci-lint
|
|
# config verify` rejects outright. Their revive settings and test exclusions
|
|
# were therefore not in effect. Consolidating fixes that too.
|
|
#
|
|
# Per-repo variation does NOT belong here -- this file must stay byte-identical
|
|
# across adopters so the gate is a checksum, not a diff a human has to judge.
|
|
# A repo with a genuine exception (agentsquad excludes its seeded-bug fixture
|
|
# tree, which exists precisely to contain broken code) keeps its own config and
|
|
# is not listed as an adopter.
|
|
|
|
version: "2"
|
|
|
|
linters:
|
|
default: none
|
|
enable:
|
|
- errcheck # unchecked errors
|
|
- govet # suspicious constructs
|
|
- staticcheck # broad static analysis
|
|
- unused # unused code
|
|
- misspell # spelling
|
|
- revive # idiomatic Go
|
|
- exhaustive # exhaustive switch on enums
|
|
|
|
settings:
|
|
revive:
|
|
rules:
|
|
- name: var-naming
|
|
severity: warning
|
|
|
|
exclusions:
|
|
rules:
|
|
# Test files: table-driven cases legitimately carry unused struct fields
|
|
# and long functions.
|
|
- path: _test\.go
|
|
linters:
|
|
- revive
|
|
- unused
|
|
|
|
formatters:
|
|
enable:
|
|
- gofumpt
|