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 termdown at 1.11.0 #23403
Feat termdown at 1.11.0 #23403
Conversation
@clemensmanert, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer. |
name = "termdown-${version}"; | ||
version = "1.11.0"; | ||
|
||
src = pkgs.fetchFromGitHub { |
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.
Shouldn't need src = pkgs.fetchFromGitHub, can simply be src = fetchFromGitHub
|
||
propagatedBuildInputs = [ dateutil click pyfiglet ]; | ||
|
||
meta = { |
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 do meta = with stdenv.lib; {
That way you don't have to write stdenv.lib in licenses and platforms
e026c94
to
7a46f15
Compare
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.
@@ -0,0 +1,27 @@ | |||
{ pkgs, stdenv, fetchFromGitHub, buildPythonApplication, |
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 pass in pkgs
, since you've passed in all your dependencies.
@@ -25935,6 +25935,8 @@ in { | |||
}; | |||
}; | |||
|
|||
termdown = callPackage ../applications/misc/termdown {}; |
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.
Python applications are not called via python-packages.nix
but directly from all-packages.nix
.
version = "1.11.0"; | ||
|
||
src = fetchFromGitHub { | ||
rev = "d1e3504e02ad49013595112cb03fbf175822e58d"; |
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.
indentation
propagatedBuildInputs = [ dateutil click pyfiglet ]; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Starts a countdown to or from TIMESPEC"; |
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.
indentation
Fixed the style issues and pushed |
Motivation for this change
Init termdown 1.11.0
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)