Skip to content

chore: remove releaser v2 flow and drop v1 naming - #27421

Merged
f0ssel merged 2 commits into
mainfrom
remove-releaser-v2
Jul 24, 2026
Merged

f0ssel merged 2 commits into
mainfrom
remove-releaser-v2

Conversation

@f0ssel

@f0ssel f0ssel commented Jul 22, 2026

Copy link
Copy Markdown
Member

Summary

Removes the GitHub Actions-driven releaser v2 pipeline so the interactive release wizard is the only release path, and drops the v1 naming now that it is the sole implementation.

Changes

  • Delete .github/workflows/tag-and-release.yaml (the v2 workflow).
  • Delete scripts/releaser/v2/.
  • Move scripts/releaser/v1/ into scripts/releaser/ as package main.
  • Rewrite scripts/releaser/main.go to a single wizard command: drop the --legacy flag and the v2 rc/branch/release subcommands and hidden CI compat commands. --dry-run is preserved.
  • Update scripts/release.sh to run go run ./scripts/releaser "$@" (no --legacy).

The legacy release.yaml workflow (triggered by scripts/release.sh) is unchanged and remains the release pipeline.

Validation

  • go build ./scripts/releaser/...
  • go test ./scripts/releaser/...
  • go vet + golangci-lint run ./scripts/releaser/...
  • gofmt -l clean

Note

The GPG signing key check removal is handled in a stacked follow-up PR based on this branch.

Implementation plan
  • v2 flow = scripts/releaser/v2/ + .github/workflows/tag-and-release.yaml (uses go run ./scripts/releaser prepare-release|generate-notes). v2 was imported only by main.go; the workflow was referenced nowhere else.
  • v1 flow = interactive wizard in scripts/releaser/v1/, reached via --legacy, driving release.yaml (triggered by scripts/release.sh).
  • No docs/ referenced the releaser tool or these workflows.
  • Steps: delete the v2 workflow and package; move v1/* up to scripts/releaser/ (package main, including test files); rewrite main.go to a single wizard command; update release.sh.

Generated by Coder Agents on behalf of @f0ssel.

@f0ssel f0ssel changed the title chore(scripts): remove releaser v2 flow and drop v1 naming chore: remove releaser v2 flow and drop v1 naming Jul 23, 2026
@f0ssel
f0ssel force-pushed the remove-releaser-v2 branch from 3e31eb1 to 0a24a07 Compare July 23, 2026 17:50
@f0ssel
f0ssel marked this pull request as ready for review July 23, 2026 18:36
@f0ssel
f0ssel requested a review from jdomeracki-coder as a code owner July 23, 2026 18:36
Make the interactive release wizard the only release path.

- Delete .github/workflows/tag-and-release.yaml and scripts/releaser/v2.
- Move scripts/releaser/v1 into scripts/releaser (package main) and drop
  the --legacy flag and v2 subcommands from main.go.
- Update scripts/release.sh to invoke the wizard directly.
@f0ssel
f0ssel force-pushed the remove-releaser-v2 branch from 0a24a07 to bb1f2d6 Compare July 23, 2026 18:43
@f0ssel
f0ssel requested a review from mtojek July 23, 2026 18:45
@f0ssel

f0ssel commented Jul 23, 2026

Copy link
Copy Markdown
Member Author

@mtojek I originally went down the road of making this a github action flow and the UX was just not good. We never used this outside of testing so it's OK to rip out.

## Summary

Removes the GPG signing key check from the release wizard. The preflight
check warned and prompted "Continue without signing?" when GPG signing
was not configured, and the same `gpgConfigured` value gated whether the
release tag was signed. Both are removed; release tags are now plain
annotated tags (`git tag -a`).

## Changes

- Remove the GPG preflight check and `gpgConfigured` plumbing from
`run.go` and `runRelease` in `release.go`.
- Drop the `sign` parameter from `ReleaseExecutor.CreateTag` and both
implementations (`liveExecutor` no longer appends `-s`).

## Validation

- `go build ./scripts/releaser/...`
- `go test ./scripts/releaser/...`
- `go vet` + `golangci-lint run ./scripts/releaser/...`
- `gofmt -l` clean

> [!IMPORTANT]
> Stacked on top of #27421 (base branch `remove-releaser-v2`).
Review/merge that PR first, then retarget this one to `main`.

---
Generated by Coder Agents on behalf of @f0ssel.
@f0ssel
f0ssel merged commit 591f357 into main Jul 24, 2026
24 checks passed
@f0ssel
f0ssel deleted the remove-releaser-v2 branch July 24, 2026 16:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants