Repository navigation
ci: add Go fmt/vet/test workflow - #4
Merged
Merged
Conversation
Gate the Go server on every push to main and every PR with the same checks AGENTS.md requires before a commit: gofmt cleanliness, go vet, and the race-enabled test suite. Pure-Go build (no CGO), so ubuntu-latest suffices; the Go version tracks go.mod. Concurrency cancels superseded runs. Also gofmt-normalise one pre-existing alignment nit in metrics_test.go so the new gofmt gate is green from the first run.
This was referenced Sep 26, 2026
tarakanof
added a commit
that referenced
this pull request
Oct 6, 2026
Reviewed: independent Opus review #305 (comment); fixes in 1cd7e74 + b5e5b87; re-check #305 (comment) (no blockers, 6/7 verified, #4 partial -> #308). CI green.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/ci.ymlrunning gofmt, go vet, and go test ./... -race on every push tomainand every PR — the checks AGENTS.md already mandates before a commit.ubuntu-latest; Go version tracksgo.mod;concurrencycancels superseded runs.cmd/ember/metrics_test.goso the new gofmt gate passes from the first run.Test Plan
gofmt -l .clean,go vet ./...clean,go test ./... -racegreen locallyGo (fmt, vet, test -race)check runs green on this PR🤖 Generated with Claude Code