Skip to content

INT-904: Modernize generated Homebrew install steps - #46

Merged
rianjs merged 1 commit into
mainfrom
rianjs/INT-904/homebrew-install-steps
Sep 22, 2026
Merged

rianjs merged 1 commit into
mainfrom
rianjs/INT-904/homebrew-install-steps

Conversation

@rianjs

@rianjs rianjs commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

[INT-904]

Summary

  • normalize GoReleaser's deprecated preflight/postflight block names before publishing casks
  • translate legacy system_command calls and staged-path interpolation to Homebrew's install-step DSL
  • derive canonical and alias casks from the same normalized file
  • add portable regression and idempotence tests

Verification

  • bash actions/homebrew-alias/test_modernize.sh
  • bash actions/homebrew-alias/test_alias.sh
  • real GoReleaser snapshot casks from cpm, hubspot-cli, and salesforce-cli normalize successfully
  • Homebrew Cask/InstallSteps style check passes on those real outputs

Rollout

After merge, advance the movable v1 tag to the merge commit before merging the dependent legacy publisher PRs.

@rianjs
rianjs merged commit eddaaa3 into main Sep 22, 2026
21 checks passed
@rianjs
rianjs deleted the rianjs/INT-904/homebrew-install-steps branch September 22, 2026 09:51
rianjs added a commit to open-cli-collective/cpm that referenced this pull request Sep 22, 2026
## [INT-904]

### Summary
- stop GoReleaser from pushing the cask directly to the tap
- render the cask with skip_upload and publish it through the shared
Homebrew action
- emit the current install-step run DSL in the source configuration

### Verification
- goreleaser check
- actionlint .github/workflows/release.yml
- PR_NUMBER=904 goreleaser release --snapshot --clean
- generated cask passes the shared normalizer and Homebrew
Cask/InstallSteps style check

### Dependency
Merge open-cli-collective/.github#46 (or its resulting PR number) and
advance the shared action's v1 tag before marking this ready.
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