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
skaffold: 0.16.0 -> 0.18.0 #50922
skaffold: 0.16.0 -> 0.18.0 #50922
Conversation
@GrahamcOfBorg build skaffold |
Success on aarch64-linux (full log) Attempted: skaffold Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: skaffold Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: skaffold Partial log (click to expand)
|
# rev is the 0.16.0 commit, mainly for skaffold version command output | ||
rev = "78e443973ee7475ee66d227431596351cf5e2caf"; | ||
rev = "34651689be78b2c6bcfbace5072b00b93661f895"; |
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.
Presumably the comment is outdated? Why is the commit hash used instead of a version tag?
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.
Ah, the comment should say 0.18.0
indeed… The main reason it is there is to put the hash in the skaffold version
(https://github.com/NixOS/nixpkgs/pull/50922/files#diff-dc620cbbae6ca2d4b7cc22b6447bf85eR15)
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.
Ah I see. A nicer way to do this may be to use git rev-parse HEAD
to get the current commit hash. For that you'd need to switch from fetchFromGitHub
to fetchgit
with the leaveDotGit
option though to leave the git meta info intact.
If you don't want to do that, I'm fine with merging as-is. Let me know.
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.
@timokau Ah, wasn't sure how simple to make it work with buildGoPackage
. I feel like we can merge this one as is and think to enhance it as a follow-up 👼
Signed-off-by: Vincent Demeester <vincent@sbr.pm>
3fe8200
to
5ce7778
Compare
Signed-off-by: Vincent Demeester vincent@sbr.pm
Motivation for this change
Bump
skaffold
to the most recent versionThings done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)