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
gitAndTools.git-when-merged: init at 1.2.0 #107012
Conversation
Result of 1 package built:
|
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.
seems like a neat tool! some small comments
@@ -0,0 +1,38 @@ | |||
{ stdenv, fetchFromGitHub, python }: |
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.
I'm not sure we want to add new dependencies on python
(which is python2
), does this tool also work with python3
?
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.
Works fine with Python3! Thanks for catching that.
sha256 = "0sw98gmsnd4iki9fx455jga9m80bxvvfgys8i1r2fc7d5whc2qa6"; | ||
}; | ||
|
||
buildInputs = [python]; |
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.
(I think we typically use spaces after brackets)
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.
fixed with nixfmt
|
||
src = fetchFromGitHub { | ||
owner = "mhagger"; | ||
repo = "${pname}"; |
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.
repo = "${pname}"; | |
repo = pname; |
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.
👍
src = fetchFromGitHub { | ||
owner = "mhagger"; | ||
repo = "${pname}"; | ||
rev = "refs/tags/v${version}"; |
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.
maybe
rev = "refs/tags/v${version}"; | |
rev = version; |
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.
that works indeed!
0abcb7f
to
d5f9b8b
Compare
Result of 1 package built:
|
pkgs/applications/version-management/git-and-tools/git-when-merged/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/version-management/git-and-tools/git-when-merged/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/version-management/git-and-tools/git-when-merged/default.nix
Show resolved
Hide resolved
d5f9b8b
to
ffcc723
Compare
Motivation for this change
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)