-
Enforce blocking golangci-lint in CI and clear lint backlog (#2996)
frostbyte_neo 发布于
2026-06-22 18:48:09 +00:00 | 800 次提交 在此版本后已推送到 master- lint: clear the golangci-lint backlog and enforce a blocking lint in CI
Fixes #2988. Brings 'golangci-lint run ./...' to zero issues (was ~373):
- errcheck: explicitly ignore fire-and-forget calls with '_ =' (and a small
errcheck.exclude-functions list for response writes — json Encoder.Encode,
http ResponseWriter.Write, fmt.Fprint*); genuine cases handled. - unused: remove dead code (unexported decls and dead test helpers) and the
imports they orphaned. - staticcheck: ST1005 error strings, ST1016 receiver names, S1000/S1017/S1019/
S1023 simplifications, SA4004/SA4006/SA4010 dead code, SA1021 net.IP.Equal,
SA6002 (store *[]byte in sync.Pool). - govet: fix a context leak (lostcancel) in internal/util/mdns and move
t.Fatal/Fatalf out of goroutines (testinggoroutine) in tests. - ineffassign, unconvert: mechanical fixes.
CI: the Lint workflow now runs a blocking full-tree 'golangci-lint run' on
pushes and PRs (dropped only-new-issues now that the tree is clean).Verified: go build, go vet, test compilation, and unit tests for the
behaviourally-touched packages all pass.Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01CmdEY7pYmV5zzwCjNJ4ykL- website: emit go-source meta for /vN paths
pkg.go.dev uses go-source to link to the right files and lines. The /vN
go-get response now returns go-source alongside go-import. Does not affect
go install resolution.
Co-authored-by: Claude noreply@anthropic.com
下载附件