Disable deprecated gomodguard linter (closes #208) #209

Closed
clawbot wants to merge 1 commits from issue-208-disable-gomodguard into next
Collaborator

Adds gomodguard to linters.disable in .golangci.yml, next to wsl. Because the config uses default: all, golangci-lint turned on the deprecated gomodguard as well as its replacement gomodguard_v2, and printed a deprecation warning on every lint run. gomodguard_v2 still runs through default: all. The pinned linter image is unchanged.

Resolves #208.

make check is green, and its lint step ran without cache and printed no deprecation warning. make fmt fails on the host because prettier is not installed there. That failure already existed and is unrelated to this change, and the YAML edit needs no formatting.

Model: opus-5-5

Adds `gomodguard` to `linters.disable` in `.golangci.yml`, next to `wsl`. Because the config uses `default: all`, golangci-lint turned on the deprecated `gomodguard` as well as its replacement `gomodguard_v2`, and printed a deprecation warning on every lint run. `gomodguard_v2` still runs through `default: all`. The pinned linter image is unchanged. Resolves https://git.eeqj.de/sneak/upaas/issues/208. `make check` is green, and its lint step ran without cache and printed no deprecation warning. `make fmt` fails on the host because `prettier` is not installed there. That failure already existed and is unrelated to this change, and the YAML edit needs no formatting. Model: opus-5-5
clawbot added the needs-review label 2026-09-23 01:29:51 +02:00
clawbot self-assigned this 2026-09-23 01:29:51 +02:00
clawbot added 1 commit 2026-09-23 01:29:51 +02:00
Disable deprecated gomodguard linter (closes #208)
Check / check (pull_request) Skipped
50e226e3d4
With default: all, golangci-lint still enables gomodguard next to its
replacement gomodguard_v2 and warns on every lint run. Disable it the
same way wsl is disabled; gomodguard_v2 stays enabled.

Model: opus-5-5
clawbot added needs-rework and removed needs-review labels 2026-09-23 01:35:09 +02:00
Author
Collaborator

needs-rework: this hand-edits the vendored .golangci.yml, which the 2026-09-22 ruling forbids. Rework it into the full re-vendor defined in #208 (comment).

Model: opus-5-5

needs-rework: this hand-edits the vendored `.golangci.yml`, which the 2026-09-22 ruling forbids. Rework it into the full re-vendor defined in https://git.eeqj.de/sneak/upaas/issues/208#issuecomment-100306. Model: opus-5-5
Author
Collaborator

Closed unmerged: #208 is rescoped to the full re-vendor of the canonical .golangci.yml, which goes in a fresh PR.

Model: opus-5-5

Closed unmerged: https://git.eeqj.de/sneak/upaas/issues/208 is rescoped to the full re-vendor of the canonical `.golangci.yml`, which goes in a fresh PR. Model: opus-5-5
clawbot removed the needs-rework label 2026-09-23 01:35:56 +02:00
clawbot closed this pull request 2026-09-23 01:35:56 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.