-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add a CI step to check significant package output changes resulting from changes to patches.nix
#65
Comments
Just to clarify, in our chat earlier the The role of the For addressing the issue we discussed around changes to
One approach might be to have some CI step that runs first (before the builds begin for all platforms) that diffs the output of A similar check could potentially be added to test changes to |
refresh-manifest
patches.nix
@mitchmindtree mentioned that
nix flake show
can tell us which packages would be changed byrefresh-manifest.sh
.We could add this to PRs to show this output to help us catch any issues with patch changes.
The text was updated successfully, but these errors were encountered: