Skip to content

ci: add gofmt + go vet lint gate to Run Tests - #36

Merged
vks-team merged 1 commit into
mainfrom
ci/lint-gate
Jul 7, 2026
Merged

ci: add gofmt + go vet lint gate to Run Tests#36
vks-team merged 1 commit into
mainfrom
ci/lint-gate

Conversation

@vks-team

@vks-team vks-team commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Item #3 of the CLI-flow alignment. Adds a lightweight lint job to run-tests.yml:

  • gofmt formatting check (fails if any file isn't gofmt-formatted)
  • go vet across the module

Both already pass repo-wide (verified), so this just locks in the current state as a per-PR gate. CI previously only built + tested; there was no style/vet gate.

Kept deliberately conservative — a fuller golangci-lint (errcheck/staticcheck/…) can follow once the team triages its findings on this never-linted codebase (adding it blind would flood).

Branch protection

After merge, a new check context lint appears — add it to main's required status checks alongside the build (...) legs and title.

Plan status (final)

Add a lightweight `lint` job (gofmt formatting check + go vet) to run-tests.yml
so style/vet regressions are caught on every PR. Both already pass repo-wide;
this is the conservative first lint gate — a fuller golangci-lint can follow once
the team triages its findings.
@vks-team
vks-team merged commit 69fa1a0 into main Jul 7, 2026
6 checks passed
@vks-team
vks-team deleted the ci/lint-gate branch July 25, 2026 15:10
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