Skip to content

chore(scripts/releaser): remove GPG signing key check - #27422

Merged
f0ssel merged 1 commit into
remove-releaser-v2from
remove-releaser-gpg-check
Jul 24, 2026
Merged

f0ssel merged 1 commit into
remove-releaser-v2from
remove-releaser-gpg-check

Conversation

@f0ssel

@f0ssel f0ssel commented Jul 22, 2026

Copy link
Copy Markdown
Member

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 force-pushed the remove-releaser-v2 branch from 3e31eb1 to 0a24a07 Compare July 23, 2026 17:50
@f0ssel
f0ssel force-pushed the remove-releaser-gpg-check branch from f735fbf to 6e3d2a5 Compare July 23, 2026 17:50
@f0ssel
f0ssel marked this pull request as ready for review July 23, 2026 18:36
Drop the preflight check that warned and prompted when GPG signing was
not configured, along with the conditional tag signing it gated. Release
tags are now plain annotated tags (git tag -a).

- Remove the GPG check and gpgConfigured plumbing from run.go/release.go.
- Drop the sign parameter from ReleaseExecutor.CreateTag and its
  implementations.
@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 jdomeracki-coder as a code owner July 23, 2026 18:43
@f0ssel
f0ssel force-pushed the remove-releaser-gpg-check branch from 6e3d2a5 to c93160c Compare July 23, 2026 18:43
@f0ssel
f0ssel requested a review from mtojek July 23, 2026 18:44

@BobbyHo BobbyHo 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.

lgtm

@f0ssel
f0ssel merged commit a81dd83 into remove-releaser-v2 Jul 24, 2026
28 checks passed
@f0ssel
f0ssel deleted the remove-releaser-gpg-check branch July 24, 2026 16:33
@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