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

caskroom: add, use any_casks_installed?. #13067

Merged
merged 1 commit into from
Mar 30, 2022
Merged

Conversation

MikeMcQuaid
Copy link
Member

Add a new method for the two places we're checking if we have any casks installed. This is much more performance and user friendly than attempting to load every cask.

Fixes #13027

Add a new method for the two places we're checking if we have any
casks installed. This is much more performance and user friendly than
attempting to load every cask.

Fixes #13027
@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Mar 30, 2022
@BrewTestBot
Copy link
Member

Review period skipped due to critical label.

@MikeMcQuaid MikeMcQuaid merged commit b2f3081 into master Mar 30, 2022
@MikeMcQuaid MikeMcQuaid deleted the any_casks_installed branch March 30, 2022 17:01
@github-actions github-actions bot added the outdated PR was locked due to age label Apr 30, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

brew doctor errors without giving advice about how to resolve No Cask with this name exists.
2 participants