Skip to content

fix(release): Open the cask PR from its own branch - #73

Draft
ananos wants to merge 1 commit into
mainfrom
fix/cask-pr-branch
Draft

ananos wants to merge 1 commit into
mainfrom
fix/cask-pr-branch

Conversation

@ananos

@ananos ananos commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

The first stable brig tag, v0.2.0, showed that repository.branch: main under homebrew_casks does not mean what the comment beside it says. It is the branch goreleaser pushes the cask to, and the head of the pull request it opens. Set to main, goreleaser pushes straight to the tap's main and then has nothing to open a PR from. On brig the tap's DCO ruleset refused that push with a 409 and the release run failed. hull carries the same line, so its first stable tag would fail the same way.

The branch is now hull-{{ .Version }}, with main as the PR base. brig shipped the same change in brig-sh/brig@131e3bc, and its second v0.2.0 run opened brig-sh/homebrew-brig#11 from brig-0.2.0 as intended.

Related issues

Refs brig-sh/homebrew-brig#11.

Changes

  • .goreleaser.yaml: homebrew_casks.repository.branch is hull-{{ .Version }} and pull_request.base.branch is main. goreleaser check passes.

Checklist

  • make test passes No Go, Swift or Rust code changed.
  • make macos builds hull, vz-runner and hvi, if Go, Swift or Rust code changed
  • I have added or updated tests covering the change Verified by the equivalent change on brig's v0.2.0 release run.
  • I have run the e2e harnesses (test/*.py) for changes touching boot, console, or the run path
  • I have updated the affected docs (README, docs/): docs/releasing.md already describes the PR path, no change needed.

`repository.branch` under homebrew_casks is the branch goreleaser pushes
the cask to, and the head of the pull request it opens. Set to main, the
first stable tag pushes straight to the tap's main and then has nothing to
open a PR from. brig's v0.2.0 run did exactly that on 2026-09-15, and the
tap's DCO ruleset refused the push with a 409. hull has the same line and
would fail the same way on its first stable tag.

The branch is now hull-{{ .Version }}, with main as the PR base, which is
what the comment beside it already described.

Signed-off-by: Anastassios Nanos <ananos@nofire.ai>
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