Skip to content

ci(release): run the checks before a tag publishes anything - #87

Merged
reachjalil merged 1 commit into
pre-releasefrom
ci/release-runs-checks-first
Nov 11, 2026
Merged

reachjalil merged 1 commit into
pre-releasefrom
ci/release-runs-checks-first

Conversation

@reachjalil

Copy link
Copy Markdown
Member

What is this about?

The release workflow runs the checks before it publishes anything.

What problem does it solve?

release.yml starts on any v* tag and went straight to building and publishing binaries, the chart and the images. A tag can be pushed on any commit, so a release could be published from code CI never checked.

How is it implemented?

A new verify job runs make fmt-check vet test experimental and make helm-lint. The binaries and images jobs need it, so a failing check publishes nothing.

It does not check which branch the tag is on, and it does not run the SDK tests; both could be added.

How was it tested?

  • The YAML parses, and the three jobs show verify with no needs and binaries/images needing it.
  • The make targets it calls pass locally.
  • Not verified: the workflow itself. It runs only when a tag is pushed, so its first real run is the next release tag.

Checklist

  • make verify passes (no code changed)
  • Commits are signed off (git commit -s)
  • API changes are reflected in api/openapi.yaml, both SDKs and docs/feature-parity.md (no API change)
  • User-visible behaviour is documented (none)

🤖 Generated with Claude Code

The release workflow starts on any `v*` tag and went straight to building
and publishing binaries, the chart and the images. A tag can be pushed on
any commit, so a release could be published from code CI never checked.

A `verify` job now runs gofmt, vet, the Go tests (default and experimental
builds) and the Helm lint, and the `binaries` and `images` jobs need it.

Not verified: the workflow runs only when a tag is pushed. The YAML parses
and the make targets it calls pass locally.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
Signed-off-by: Jalil <reachjalil@gmail.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI passed at 254ceda.

@reachjalil
reachjalil merged commit 2bdeb69 into pre-release Nov 11, 2026
17 checks passed
@reachjalil
reachjalil deleted the ci/release-runs-checks-first branch November 11, 2026 00:13
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