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
k9s: Init at 0.7.13 #65432
k9s: Init at 0.7.13 #65432
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.
Thanks for this! Please fix the first line.
@GrahamcOfBorg build kubernetes-k9s |
@GrahamcOfBorg build kubernetes-k9s |
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.
Why don't we just call it k9s
since there's no conflict?
rebased & squashed, applied all suggestions |
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
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.
ARCHITECTURE = [ "NixOs x86_64" ]
Reviewed points
- package path fits guidelines
- package name fits guidelines
- package version fits guidelines
- package build on ARCHITECTURE
- executables tested on ARCHITECTURE
-
meta.description
is set and fits guidelines -
meta.license
fits upstream license -
meta.platforms
is set -
meta.maintainers
is set - build time only dependencies are declared in
nativeBuildInputs
- source is fetched using the appropriate function
- phases are respected
- patches that are remotely available are fetched with
fetchpatch
Comments
Did not know about the tool before. Looks pretty good!
@GrahamcOfBorg build k9s |
Thank you everyone for taking look at this! |
Motivation for this change
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)