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
onefetch: init at 1.7.0 #72254
onefetch: init at 1.7.0 #72254
Conversation
@@ -3484,6 +3484,16 @@ | |||
github = "klntsky"; | |||
githubId = 18447310; | |||
}; | |||
kloenk = { |
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.
Do you mind creating a separate commit for adding an user?
, makeRustPlatform | ||
, fetchFromGitHub | ||
, lib | ||
, ... |
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.
What is the purpose?
|
||
meta = with lib; { | ||
homepage = https://github.com/o2sh/onefetch; | ||
description = ''Displays information about your Git project directly on your terminal''; |
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 don't use ''
if no multi line.
CARGO_HOME = "$(mktemp -d cargo-home.XXX)"; | ||
|
||
meta = with lib; { | ||
homepage = https://github.com/o2sh/onefetch; |
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 use quotes if possible.
}; | ||
cargoSha256 = "0cpfjxn6nqsrnmgvjzr17n6xbbyfl9c91d5kbrmxb6jyig98k6aq"; | ||
buildInputs = [ ]; | ||
CARGO_HOME = "$(mktemp -d cargo-home.XXX)"; |
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.
Isn't this created by fetchCargo?
sha256 = "1p16mg4ak9ppx3y11l3r4y6356drwhnmrlxsaqx01n53ii5ij9kg"; | ||
}; | ||
cargoSha256 = "0cpfjxn6nqsrnmgvjzr17n6xbbyfl9c91d5kbrmxb6jyig98k6aq"; | ||
buildInputs = [ ]; |
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.
What is the reason to have an empty buldInputs?
, makeRustPlatform | ||
, fetchFromGitHub | ||
, lib | ||
, ... |
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 needed
, ... |
I hade not so much time, because of school, so I will close this PR in favor of #74191 |
Motivation for this change
Adds onefetch for git repos analysis
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)