Skip to content

ci: align workflows with labstack/echo - #151

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

vishr merged 2 commits into
masterfrom
ci/standardize-workflows

Conversation

@vishr

@vishr vishr commented Oct 1, 2026

Copy link
Copy Markdown
Member

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

  • Naming: echo-contrib.yml is now ci.yml ("CI"), the most common name for a main build-and-test workflow.
  • Go versions: tests run on Go 1.25, 1.26 and 1.27; coverage and benchmarks use 1.27.
  • Supply chain: actions are pinned to commits (checkout v6, setup-go v6, codecov v7, cache v6.1.0).
  • 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 master with itself. It also uses -benchtime=100ms, so it takes minutes instead of about 20.
  • Concurrency: superseded pull request runs are cancelled; runs on master always finish.

https://claude.ai/code/session_01QKDYQr53zNKkR7nif2CAAq

vishr added 2 commits October 1, 2026 15:18
- Rename the test workflow to ci.yml ("CI").
- Test with Go 1.25, 1.26 and 1.27, and run coverage and benchmarks on 1.27.
- Pin actions to commits (checkout v6, setup-go v6, codecov v7, cache v6.1.0).
- 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 master with itself), with -benchtime=100ms so it takes
  minutes instead of about 20.
- Cancel superseded pull request runs.

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 1c1e57f into master Oct 1, 2026
10 checks passed
@vishr
vishr deleted the ci/standardize-workflows branch October 1, 2026 22:30
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