Use the canonical golangci-lint config (closes #66) #97

Merged
clawbot merged 1 commits from issue-66-gomodguard-v2 into next 2026-10-04 15:32:23 +02:00
Collaborator

Replaces .golangci.yml with the canonical file from https://git.eeqj.de/sneak/prompts/raw/branch/main/.golangci.yml, byte for byte, as REPO_POLICIES.md requires. Fixes #66.

What the canonical file changes here:

  • gomodguard is disabled and gomodguard_v2 enabled in its place, with the module block list from the Go package defaults. The deprecation warning is gone from the lint output.
  • depguard is no longer disabled. It carries one rule: net/http/httptest may not be imported by a file that is not a test. This repo does not import it anywhere.

Neither linter raised a finding, so no code changed.

Judgement call: the canonical file says a repo may add its own test-support packages to the depguard deny list. This repo has none (its in-process test helper in internal/cli/test_helpers.go imports nothing test-only), and the brief asked for no local edits, so the list is left as is.

Model: opus-5-5

Replaces `.golangci.yml` with the canonical file from https://git.eeqj.de/sneak/prompts/raw/branch/main/.golangci.yml, byte for byte, as `REPO_POLICIES.md` requires. Fixes https://git.eeqj.de/sneak/secret/issues/66. What the canonical file changes here: - `gomodguard` is disabled and `gomodguard_v2` enabled in its place, with the module block list from the Go package defaults. The deprecation warning is gone from the lint output. - `depguard` is no longer disabled. It carries one rule: `net/http/httptest` may not be imported by a file that is not a test. This repo does not import it anywhere. Neither linter raised a finding, so no code changed. Judgement call: the canonical file says a repo may add its own test-support packages to the `depguard` deny list. This repo has none (its in-process test helper in `internal/cli/test_helpers.go` imports nothing test-only), and the brief asked for no local edits, so the list is left as is. Model: opus-5-5
clawbot added the needs-review label 2026-10-04 14:50:17 +02:00
clawbot self-assigned this 2026-10-04 14:50:17 +02:00
clawbot added 1 commit 2026-10-04 14:50:18 +02:00
Replace `.golangci.yml` with the canonical file from `sneak/prompts`,
byte for byte. It runs `gomodguard_v2` in place of the deprecated
`gomodguard`, which removes the deprecation warning from every lint run,
and enables a `depguard` rule keeping `net/http/httptest` out of non-test
files. Neither raised a finding in this repo.

Model: opus-5-5
Author
Collaborator

PASS: .golangci.yml matches the canonical file byte for byte, the gomodguard deprecation warning is gone, and nothing else needed to change.

Model: opus-5-5

PASS: `.golangci.yml` matches the canonical file byte for byte, the `gomodguard` deprecation warning is gone, and nothing else needed to change. Model: opus-5-5
clawbot merged commit 5ca615a7a6 into next 2026-10-04 15:32:23 +02:00
clawbot deleted branch issue-66-gomodguard-v2 2026-10-04 15:32:23 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/secret#97