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

feat(homebrew): Add option to upgrade all casks #82

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tuxpeople
Copy link

Hi

I had an issue with upgrading as some casks weren't upgraded. I tried to use the cu plugin to upgrade all the casks, including those who wouldn't be upgraded as they are marked as "auto_upgrade". I used this command in my playbook: brew cu -a -y

But I think it should be part of this role, therefore I submit this PR. I tested it locally here.

Before the test:

$ brew outdated --cask --greedy
font-roboto (latest) != latest
font-roboto-mono (latest) != latest
webex-meetings (2211.1803.4212.2) != latest
wireshark (4.0.2) != 4.0.10

After the test:

$ brew outdated --cask --greedy
font-roboto (latest) != latest
font-roboto-mono (latest) != latest

I've no idea about the fonts. Probably there's an issue with the version being latest.

This PR adds a task to the cask part like it already exists in the brew part. It has a similar variable to enable it, and like the task in the brew part, it's disabled by default.

Br

@tuxpeople
Copy link
Author

@geerlingguy does it look okay for you?

Copy link

stale bot commented Mar 17, 2024

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

@stale stale bot added the stale label Mar 17, 2024
@tuxpeople
Copy link
Author

still thinks this is a good idea... how about you @geerlingguy

Copy link

stale bot commented Aug 7, 2024

This pull request is no longer marked for closure.

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.

1 participant