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
skopeo: 0.1.36 -> 0.1.37 #63352
skopeo: 0.1.36 -> 0.1.37 #63352
Conversation
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@GrahamcOfBorg build skopeo |
@GrahamcOfBorg test docker-tools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why the test failed, but:
- reviewed the diff and commit messages
- made sure ofBorg build succeeded for all applicable platforms
- run nix-review without any failures
- run and tested the binaries
ping @lewo @offlinehacker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skopeo works, I do get the same test failure locally though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just putting in a red review to avoid accidental merges while trying to figure out the test failure. I think it's critical enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nevermind, rebasing on top of master actually removes the error. Phew! 😁
Motivation for this change
Bump to latest release of
skopeo
.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)