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

concept2-utility: migrate from cask-drivers #146716

Merged
merged 2 commits into from
May 8, 2023
Merged

concept2-utility: migrate from cask-drivers #146716

merged 2 commits into from
May 8, 2023

Conversation

thalamus
Copy link
Contributor

@thalamus thalamus commented May 8, 2023

Migrating after Homebrew/homebrew-cask-drivers#3443 - merge with Homebrew/homebrew-cask-drivers#3454

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.
  • Checked the cask is submitted to the correct repo.
  • brew audit --new-cask <cask> worked successfully.
  • brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

@BrewTestBot BrewTestBot added new cask missing zap Cask is missing a zap stanza, please add one. labels May 8, 2023
@razvanazamfirei
Copy link
Contributor

Would you mind adding a zap stanza for this cask?

@BrewTestBot BrewTestBot removed the missing zap Cask is missing a zap stanza, please add one. label May 8, 2023
@thalamus
Copy link
Contributor Author

thalamus commented May 8, 2023

Hi @razvanazamfirei, I've added a zap stanza.

@razvanazamfirei
Copy link
Contributor

Thank you!

@razvanazamfirei razvanazamfirei enabled auto-merge (squash) May 8, 2023 14:14
@razvanazamfirei razvanazamfirei merged commit cbbc762 into Homebrew:master May 8, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants