Skip to content

ci: add lint/test on PRs and publish on release#25

Merged
FumingPower3925 merged 5 commits intomainfrom
ci/github-actions
Mar 29, 2026
Merged

ci: add lint/test on PRs and publish on release#25
FumingPower3925 merged 5 commits intomainfrom
ci/github-actions

Conversation

@FumingPower3925
Copy link
Copy Markdown
Contributor

@FumingPower3925 FumingPower3925 commented Mar 29, 2026

Summary

  • ci.yml: Runs on all PRs to main and pushes to main
    • lint job: golangci-lint via official action
    • test job: go test -race -count=1 -timeout 120s ./...
  • release.yml: Runs when a GitHub release is published
    • Verifies module (go mod verify, go build, go vet)
    • Hits the Go module proxy and checksum DB to trigger pkg.go.dev indexing

Test plan

  • YAML syntax valid
  • CI jobs run on this PR itself

- ci.yml: golangci-lint + go test -race on all PRs to main
- release.yml: verify module and trigger pkg.go.dev indexing on release
Weekly checks for Go module and GitHub Actions updates,
all grouped into a single PR per ecosystem.
@FumingPower3925 FumingPower3925 self-assigned this Mar 29, 2026
golangci-lint v1.x is built with Go 1.24 and cannot target Go 1.26.
Switch to golangci-lint-action@v9 with v2.9, matching the celeris repo.
Add .golangci.yml config (errcheck, govet, staticcheck, revive, etc.).
Also bump actions/checkout and actions/setup-go to v6.
- Fix package comment: "Package bench" → "Package loadgen" (revive)
- Rename java_like_format → javaLikeFormat (revive var-naming)
- Fix gofmt alignment in Config structs across 4 files
- Add _ = for unchecked Close/Write returns in tests (errcheck)
- Use _ for unused http.Request and callback params in tests (revive)
@FumingPower3925 FumingPower3925 merged commit 36e9406 into main Mar 29, 2026
2 checks passed
@FumingPower3925 FumingPower3925 deleted the ci/github-actions branch March 29, 2026 02:28
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