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
feat/ Added nb #100019
feat/ Added nb #100019
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.
Other than the two inline comments LGTM (none of them seems like a real issue)
src = fetchgit { | ||
url = "https://github.com/xwmx/nb.git"; | ||
rev = version; | ||
sha256 = "LXgLzO+rPCalkWBF79U2Y8r+wtTfSMMuHRYlT/wKd3g="; | ||
}; |
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 believe currently fetchFromGitHub
is preferred over `fetchgit if no submodules are needed?
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.
Interesting, where should I look to find conventions like this?
I don't have time to fix this up right now, but probably will this week (otherwise updates welcome).
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 pname and version and replace let in with rec. Also update the PR title and commit title to fit the contributing guide.
I marked this as stale due to inactivity. → More info |
nb was added in 44414b4 |
Motivation for this change
Adding
nb
to nixpkgs, since I already wrote a derivation.Very small 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)