Skip to content

ci: align workflows with labstack/echo - #43

Merged
vishr merged 2 commits into
mainfrom
ci/standardize-workflows
Oct 1, 2026
Merged

vishr merged 2 commits into
mainfrom
ci/standardize-workflows

Conversation

@vishr

@vishr vishr commented Oct 1, 2026

Copy link
Copy Markdown
Member

Brings the workflows in line with labstack/echo (labstack/echo#3139, #3140 and #3141):

  • Naming: the test workflow is now ci.yml ("CI"), the most common name for a main build-and-test workflow; the lint workflow is "Checks".
  • Go versions: tests run on Go 1.25, 1.26 and 1.27; coverage, benchmarks and checks use 1.27.
  • Build cache: setup-go's cache is keyed on go.sum only, so it is never refreshed. Each job now restores the newest actions/cache entry for its OS, Go version and job, and pushes save a new one.
  • Benchmarks: the comparison runs only on pull requests. github.base_ref is empty on pushes, so it was comparing main with itself. It also uses -benchtime=100ms, so it takes minutes instead of about 20.
  • Concurrency: superseded pull request runs are cancelled; runs on main always finish.

https://claude.ai/code/session_01QKDYQr53zNKkR7nif2CAAq

vishr added 2 commits October 1, 2026 15:17
- Rename the test workflow to ci.yml ("CI"); the lint workflow is now "Checks".
- Test with Go 1.25, 1.26 and 1.27, and run coverage, benchmarks and checks on 1.27.
- Keep the Go build cache current: restore the newest cache for each OS, Go version
  and job, and save a new one on pushes (setup-go's cache is keyed on go.sum only
  and goes stale).
- Run the benchmark comparison only on pull requests (github.base_ref is empty on
  pushes, so it compared the branch with itself), with -benchtime=100ms so it
  takes minutes instead of about 20.
- Cancel superseded pull request runs.
- Pin actions/cache to a commit, like the other actions.

Claude-Session: https://claude.ai/code/session_01QKDYQr53zNKkR7nif2CAAq
The rename to ci.yml also renamed the test step; only the workflow is CI.

Claude-Session: https://claude.ai/code/session_01QKDYQr53zNKkR7nif2CAAq
@vishr
vishr merged commit 5da5df4 into main Oct 1, 2026
11 checks passed
@vishr
vishr deleted the ci/standardize-workflows branch October 1, 2026 22: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