Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(re)add marathon2 and marathon-infinity casks #88604

Merged

Conversation

Amorymeltzer
Copy link
Contributor

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • brew cask audit --new-cask {{cask_file}} worked successfully.
  • brew cask install {{cask_file}} worked successfully.
  • brew cask uninstall {{cask_file}} worked successfully.
  • Checked the cask was not already refused.
  • Checked the cask is submitted to the correct repo.

Both were removed in analytics purges, but I propose to re-add them to match/parallel the marathon and aleph-one casks.

*marathon2: Removed in #63338 (cc482d5)
*marathon-infinity: Removed in #72428 (3d28779)

It's a little bit of a weird case, as while it is an active project with several alpha releases over the years, it was just officially updated for the first time in 5 years. All four casks are from the same repo (https://github.com/Aleph-One-Marathon/alephone), but that repo appear to meet the [https://github.com/Homebrew/homebrew-cask/blob/master/doc/faq/rejected_casks.md popularity recommendations]. I'm obviously not privy to the stats when they were purged, but I do think it makes sense to have these two for completion. Obviously happy to discuss (cc @vitorgalvao) if views differ, although I think this aligns with #63053 (comment) and #63053 (comment).

Both were removed in analytics purges, but they are being re added to match/parallel the marathon and aleph-one casks.

*marathon2: Removed in Homebrew#63338 (cc482d5)
*marathon-infinity: Removed in Homebrew#72428 (3d28779)
@vitorgalvao vitorgalvao merged commit 57e3ce8 into Homebrew:master Sep 3, 2020
Amorymeltzer added a commit to Amorymeltzer/homebrew-cask that referenced this pull request Sep 12, 2020
Was erroneously given the description of alephone in Homebrew#87436 (4002df8); this matches the descriptions given to M2:D and M:I in Homebrew#88604
@Amorymeltzer Amorymeltzer mentioned this pull request Sep 12, 2020
10 tasks
BrewTestBot pushed a commit that referenced this pull request Sep 12, 2020
Was erroneously given the description of alephone in #87436 (4002df8); this matches the descriptions given to M2:D and M:I in #88604
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.

2 participants