Skip to content

Conversation

MatthiasPortzel
Copy link

@MatthiasPortzel MatthiasPortzel commented Sep 15, 2025

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew typecheck with your changes locally?
  • Have you successfully run brew tests with your changes locally?

Fixes #20222.

I'm not attached to this change-set, feel free to modify before merging. Just wanted to put up a PR inline with what we discussed on that issue.

Cleanup has not been called at this point of course, but some dependencies, like `CaskDumper`, have instance variables cached that need to be reset after installing.
@MatthiasPortzel MatthiasPortzel marked this pull request as draft September 16, 2025 00:09
@MatthiasPortzel
Copy link
Author

Marking as draft because I'm only about 80% confident that this is correct. I haven't tested installing a full Brewfile with these changes.

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.

brew bundle handles renamed apps incorrectly
1 participant