Skip to content

executor 1.4.28 (new cask)#264168

Open
connorads wants to merge 2 commits into
Homebrew:mainfrom
connorads:executor-desktop-1.4.28
Open

executor 1.4.28 (new cask)#264168
connorads wants to merge 2 commits into
Homebrew:mainfrom
connorads:executor-desktop-1.4.28

Conversation

@connorads
Copy link
Copy Markdown
Contributor

@connorads connorads commented May 13, 2026

Adds a new cask for Executor

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

  • AI was used to generate or assist with generating this PR.

I opened the app, added some config, ran brew uninstall --cask then reinstalled and confirmed the source persisted. I then ran brew uninstall --zap --cask then reinstalled and confirmed fresh empty state. This means removing ~/.executor which is probably a bit unusual - happy to change that if desired.

@connorads connorads marked this pull request as ready for review May 13, 2026 19:53
Comment thread Casks/e/executor-desktop.rb Outdated
Maintainer review: prefer bare `executor` token. If an upstream CLI
is added to homebrew-core later it can use `executor-cli`.
@connorads connorads changed the title executor-desktop 1.4.28 (new cask) executor 1.4.28 (new cask) May 14, 2026
connorads added a commit to connorads/dotfiles that referenced this pull request May 15, 2026
Maintainer feedback on PR Homebrew/homebrew-cask#264168 clarified
the convention: bare token goes to whichever component lands in
Homebrew first. Subsequent siblings take suffixes (-cli, etc.).
Keep -desktop only for brand names ("Docker Desktop") or when a
sibling already exists in Homebrew under the bare name. An
upstream CLI not yet in Homebrew is not a reason to suffix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants