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

[PR #8708/76d0222a backport][stable-8] homebrew_cask: fix upgrade_all changed when nothing upgraded #8748

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Aug 12, 2024

This is a backport of PR #8708 as merged into main (76d0222).

SUMMARY

Fixes bug with homebrew_cask upgrade_all where the module always returns 'changed' due to change in output behavior of the brew command.

Fixes #8707

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

homebrew_cask

* homebrew_cask: fix upgrade_all changed when nothing upgraded

* Add changelog fragment

* Update changelogs/fragments/8708-homebrew_cask-fix-upgrade-all.yml

Co-authored-by: Felix Fontein <[email protected]>

* Add .strip() to upgrade output check

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: John Byrne <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 76d0222)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type) labels Aug 12, 2024
@felixfontein felixfontein merged commit d0fe612 into stable-8 Aug 12, 2024
146 checks passed
@felixfontein felixfontein deleted the patchback/backports/stable-8/76d0222a83d7986e8fa6f693e87218cd0931f835/pr-8708 branch August 12, 2024 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants