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
gcalcli: init at 3.4.0 #22725
gcalcli: init at 3.4.0 #22725
Conversation
@@ -0,0 +1,30 @@ | |||
{ stdenv, fetchFromGitHub, pkgs, lib, python, pythonPackages }: |
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.
stdenv
is not used.- never pass in
pkgs
. - pass in either
python
orpythonPackages
, no need for both. You can accesspythonPackages
viapython.pkgs
and the interpreter withpythonPackages.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.
Thanks for the feedback. I'm still new to Nix... I'll get that updated.
meta = with lib; { | ||
homepage = https://github.com/insanum/gcalcli; | ||
description = "CLI for Google Calendar"; | ||
license = licenses.mit; |
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 going to maintain 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 can.
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.
If this all looks good then I think this is ready to go.
Motivation for this change
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/
)