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
ytt: init at 0.30.0 #105071
ytt: init at 0.30.0 #105071
Conversation
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.
LGTM
2c63a28
to
800f9b7
Compare
Result of 1 package built:
|
d9229cb
to
eee9a96
Compare
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.
Please change the title and commit message to not contain a v in the version number.
Result of 1 package built:
|
Result of 1 package built:
|
Added carvel ytt 0.30.0 which is currently absent. Tests ran by checkPhase currently fail due to YAML marshalling problems, but the tool itself appears fine.
Change the title and commit. |
Congratulations on your first PR. |
Thanks for the assistance! |
https://github.com/vmware-tanzu/carvel-ytt
Motivation for this change
ytt is not already available in nixpkgs. Tests ran by checkPhase fail due to YAML marshaling issues but failures are unrelated to the result binary.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)