Skip to content

ci: align workflows with labstack/echo - #22

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".
  • Supply chain: actions are pinned to commits (checkout v6, setup-go v6, codecov v7, cache v6.1.0), and gosec is pinned to v2.29.0 instead of being installed from its master branch.
  • 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: -benchtime=100ms, so the comparison takes minutes instead of about 20. It already ran only on pull requests.
  • Concurrency: superseded pull request runs are cancelled; runs on main always finish.

The repository-specific steps stay as they were: tests against the latest Echo v5, the example build, and gosec.

https://claude.ai/code/session_01QKDYQr53zNKkR7nif2CAAq

vishr added 2 commits October 1, 2026 15:18
- Rename the test workflow to ci.yml ("CI"); the lint workflow is now "Checks".
- Pin actions to commits (checkout v6, setup-go v6, codecov v7, cache v6.1.0).
- Pin gosec to v2.29.0 instead of installing from its master branch.
- 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 benchmarks with -benchtime=100ms so the comparison takes minutes instead of
  about 20.
- Cancel superseded pull request runs.

The extra steps stay: tests against the latest Echo v5, the example build and gosec.

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 6a87d77 into main Oct 1, 2026
11 checks passed
@vishr
vishr deleted the ci/standardize-workflows branch October 1, 2026 22:29
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