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

mipony 1.12 #171990

Merged
merged 3 commits into from
Apr 23, 2024
Merged

mipony 1.12 #171990

merged 3 commits into from
Apr 23, 2024

Conversation

adithyasunil26
Copy link
Contributor

Created with brew bump-cask-pr.

@BrewTestBot BrewTestBot added bump-cask-pr PR was created using `brew bump-cask-pr` missing zap Cask is missing a zap stanza, please add one. labels Apr 23, 2024
@krehel
Copy link
Member

krehel commented Apr 23, 2024

Hi @adithyasunil26 - can you help us here with a few things?

First, can you try to add a zap for this?

Second, we're auditing all Casks using :extract_plist as a download strategy to determine if there's an alternative as part of #171006. Do you mind looking into that also?

Thank you! 🙏

@BrewTestBot BrewTestBot removed the missing zap Cask is missing a zap stanza, please add one. label Apr 23, 2024
@adithyasunil26
Copy link
Contributor Author

adithyasunil26 commented Apr 23, 2024

@krehel I have added the zap. There doesn't seem to be any alternatives to the :extract_plist strategy in case of this cask as the mac OS version is not mentioned anywhere on the website or in URLs.

@krehel
Copy link
Member

krehel commented Apr 23, 2024

Thank you @adithyasunil26!

@krehel krehel merged commit 74e344a into Homebrew:master Apr 23, 2024
9 checks passed
@adithyasunil26 adithyasunil26 deleted the bump-mipony-1.12-0 branch April 23, 2024 15:10
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-cask-pr PR was created using `brew bump-cask-pr` outdated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants