-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
idid: init at 2.1.0 #68968
idid: init at 2.1.0 #68968
Conversation
@@ -1492,6 +1492,12 @@ | |||
githubId = 49904992; | |||
name = "Dawid Sowa"; | |||
}; | |||
dbalan = { |
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 do you mind splitting this change into its separate commit?
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.
@mmahut Thanks for taking a look, I've split the change into two commits.
Adds a new package idid to nixpkgs.
let | ||
version = "2.1.0"; | ||
in | ||
|
||
mkDerivation { | ||
pname = "idid"; | ||
inherit 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.
you can write it like this:
mkDerivation rec {
pname = "idid";
version = "2.1.0";
@ajs124 told me you should package it on hackage ;) |
@Lassulus Thanks! I'll try to include it via hackage. |
Adds a new package idid to nixpkgs.
Motivation for this change
Adds a new package that I wrote and use (it was a script in my personal nix config so far)
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)Notify maintainers
cc @