Skip to content

Apply → Remove cache as a separate step - #113

Merged
revett merged 2 commits into
mainfrom
fix/111/remove-cache
Aug 26, 2026
Merged

Apply → Remove cache as a separate step#113
revett merged 2 commits into
mainfrom
fix/111/remove-cache

Conversation

@revett

@revett revett commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Resolves #111

Problem

  • Every apply run prompted "Uninstall these packages?" even when nothing was floating
  • The dry run of brew bundle cleanup also lists cache files it would remove, and these were being shown as packages
  • Confirming then printed noisy Removing: ... and freed ...MB output under the package removal step

Changes

  • Only entries from the package sections of the dry run (formulae, casks, taps) are shown and prompted for
  • Added a Removing cache step that runs brew cleanup unconditionally after install/upgrade and the final check
  • Cache output is captured and shown via clack only when there is something to report, so the step stays tidy when idle

Why

  • Removing cache is housekeeping, not a destructive change, so it should never gate on a prompt
  • Running it last means old versions left behind by upgrades are cleaned up in the same run

@revett
revett merged commit e225148 into main Aug 26, 2026
2 checks passed
@revett
revett deleted the fix/111/remove-cache branch August 26, 2026 12:08
@revett revett mentioned this pull request Aug 26, 2026
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.

Remove cache as part of cleanup

1 participant