Skip to content

Pin lint tooling and local checks#89

Open
marcus wants to merge 1 commit intomainfrom
lint-fix-tooling-consistency-v2
Open

Pin lint tooling and local checks#89
marcus wants to merge 1 commit intomainfrom
lint-fix-tooling-consistency-v2

Conversation

@marcus
Copy link
Copy Markdown
Owner

@marcus marcus commented Mar 31, 2026

Summary

  • pin local golangci-lint runs to v1.64.8 via a repo-local install target
  • replace ad hoc formatting/vet hook logic with shared make targets
  • make CI use go.mod's Go version and the same fmt/vet/lint entry points

Verification

  • make fmt-check
  • make vet
  • make lint
  • ./scripts/pre-commit.sh
  • .git/hooks/pre-commit
  • gofmt -l $(git ls-files '*.go')
  • ./bin/golangci-lint version --format short
  • go test ./... (fails only on pre-existing internal/providers/copilot_test.go:152)

Automated by nightshift

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant