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

terminology: mark no zap required #191570

Merged
merged 2 commits into from
Nov 11, 2024
Merged

terminology: mark no zap required #191570

merged 2 commits into from
Nov 11, 2024

Conversation

krehel
Copy link
Member

@krehel krehel commented Nov 11, 2024

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

In the following questions <cask> is the token of the cask you're submitting.

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


Application installs a dictionary which is removed on an uninstall. No other files are created, so therefore marking as no zap required.

@BrewTestBot BrewTestBot added missing description Cask is missing a description, please add one. extract_plist livecheck automerge-skip and removed missing description Cask is missing a description, please add one. labels Nov 11, 2024
@krehel
Copy link
Member Author

krehel commented Nov 11, 2024

Thanks @bevanjkay!

@khipp khipp merged commit f1241df into master Nov 11, 2024
8 of 10 checks passed
@khipp khipp deleted the terminology-no-zap branch November 11, 2024 09:59
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