Update golangci-lint to v2.12.2 with canonical config #4

clawbot · 2026-08-07T19:11:14+02:00

clawbot commented

2026-08-07 19:11:14 +02:00

Updates the linter toolchain and config:

.golangci.yml replaced with the canonical v2-schema config. Linter settings now live under linters.settings (the previous top-level linters-settings block was silently ignored by golangci-lint v2, so thresholds like lll: 88 were not being applied). The obsolete issues.exclude-use-default key is dropped.
golangci-lint bumped from the v2.10.1-era commit pin to @v2.12.2 in Dockerfile and script/bootstrap; pin date comments refreshed to 2026-08-07.
Lint fixes: 15 lll findings in attrsum.go (long function signatures, one Fprintf format string, one flag description, and one trailing nolint comment line) resolved by wrapping lines; no behavior changes. The nolint:gosec directive moved to its own line, unchanged in substance.
TODO.md Completed Steps updated.

make check (fmt-check, vet, lint, tests) passes.

Updates the linter toolchain and config: - `.golangci.yml` replaced with the canonical v2-schema config. Linter settings now live under `linters.settings` (the previous top-level `linters-settings` block was silently ignored by golangci-lint v2, so thresholds like `lll: 88` were not being applied). The obsolete `issues.exclude-use-default` key is dropped. - golangci-lint bumped from the v2.10.1-era commit pin to `@v2.12.2` in `Dockerfile` and `script/bootstrap`; pin date comments refreshed to 2026-08-07. - Lint fixes: 15 `lll` findings in `attrsum.go` (long function signatures, one `Fprintf` format string, one flag description, and one trailing `nolint` comment line) resolved by wrapping lines; no behavior changes. The `nolint:gosec` directive moved to its own line, unchanged in substance. - `TODO.md` Completed Steps updated. `make check` (fmt-check, vet, lint, tests) passes.

clawbot added 1 commit 2026-08-07 19:11:15 +02:00

Update golangci-lint to v2.12.2 with canonical config

check / check (push) Successful in 32s

Details

a5d4cd6c13

- Replace .golangci.yml with the canonical v2-schema config: linter
  settings move under linters.settings (the previous top-level
  linters-settings block was ignored by golangci-lint v2, so the
  configured thresholds were not applied) and the obsolete
  issues.exclude-use-default key is dropped.
- Bump golangci-lint from the v2.10.1-era commit pin to @v2.12.2 in
  Dockerfile and script/bootstrap; refresh pin date comments.
- Wrap long lines in attrsum.go to satisfy the now-effective lll
  limit of 88 columns (15 findings); move one nolint:gosec directive
  to its own line.
- Record the change in TODO.md Completed Steps.

clawbot self-assigned this 2026-08-10 15:46:33 +02:00

check / check (push) Successful in 32s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin golangci-v2.12.2:golangci-v2.12.2

git checkout golangci-v2.12.2

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sneak/attrsum#4