Skip to content

xdeck 3.0 (new cask)#258760

Open
morishin wants to merge 3 commits intoHomebrew:mainfrom
morishin:add-xdeck
Open

xdeck 3.0 (new cask)#258760
morishin wants to merge 3 commits intoHomebrew:mainfrom
morishin:add-xdeck

Conversation

@morishin
Copy link
Copy Markdown


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 xdeck worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask xdeck worked successfully.
  • brew uninstall --cask xdeck worked successfully.

  • AI was used to generate or assist with generating this PR. Claude Code was used to generate the cask formula and run the audit/style checks. The install and uninstall were verified on the author's machine.

@BrewTestBot BrewTestBot added new cask missing zap Cask is missing a zap stanza, please add one. labels Apr 11, 2026
Copy link
Copy Markdown
Member

@bevanjkay bevanjkay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please try to add a zap stanza. The linked documentation provides some recommended methods to create it.

Casks/x/xdeck.rb Outdated
Comment on lines +10 to +14
livecheck do
url :url
strategy :github_latest
end

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The livecheck stanza is likely unnecessary, as the upstream repository does not seem to utilise GitHub pre-releases or have tags without releases. The default livecheck should suffice.

Suggested change
livecheck do
url :url
strategy :github_latest
end

@BrewTestBot BrewTestBot removed the missing zap Cask is missing a zap stanza, please add one. label Apr 13, 2026
@morishin
Copy link
Copy Markdown
Author

@bevanjkay Thanks for the review! I've removed the unnecessary livecheck stanza and added a zap stanza.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants