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
countdown: init at 1.0.0 #65411
countdown: init at 1.0.0 #65411
Conversation
@@ -0,0 +1,24 @@ | |||
{ stdenv, buildGoPackage, fetchFromGitHub }: | |||
buildGoPackage rec { | |||
name = "countdown-${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.
Please consider using pname
.
meta = with stdenv.lib; { | ||
homepage = https://github.com/antonmedv/countdown; | ||
description = "Terminal countdown timer"; | ||
maintainers = with maintainers; [ ]; |
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.
Are you sure you don't want to add yourself as a maintainer of this package?
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've added myself as a maintainer for the package!
@GrahamcOfBorg build countdown |
Please squash your commits, thank you. @GrahamcOfBorg build countdown |
I did, but no matter what I do I fail to make my maintainer-list.nix file and have it merge successfully. My latest attempt was to checkout the relevant file from master, add my changes and then squash everything together into one commit. What would be a better way to fix this? |
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
Motivation for this change
New package for a simple, terminal based countdown program.
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)