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
Show New/Deleted Formulae/Casks from brew update
#14802
Show New/Deleted Formulae/Casks from brew update
#14802
Conversation
Review period will end on 2023-02-27 at 17:30:54 UTC. |
Idea here makes sense, yeah. 👍 |
This general approach makes sense to me. |
Review period ended. |
If we're installing from the API: we cannot use Git to check for differences in packages so instead use `{formula,cask}_names.txt` to do so. The first time this runs: we won't yet have a base state (`{formula,cask}_names.before.txt`) to compare against so we don't output anything and just copy the files for next time.
37f65b8
to
2451d01
Compare
This is now ready for review/testing. Note: because this is only changing the |
What would be the best way to test this? I checked this branch out yesterday and ran |
I'm confident enough in this to merge as-is. Should probably follow-up letting people with Feel free to revert if this seems to introduce problems. |
(Edit: Never mind, I see that this is already what #14866 does; feel free to ignore my comment here.) |
If we're installing from the API: we cannot use Git to check for differences in
packages so instead use
{formula,cask}_names.txt
to do so.The first time this runs: we won't yet have a base state
(
{formula,cask}_names.before.txt
) to compare against so we don't outputanything and just copy the files for next time.
Fixes #14706