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
pythonPackages.chevron: init at 0.13.1 #59088
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.
Just the usual packaging nitpicks.
Mainly tests need to be enabled somehow, see comments below.
I'd like to merge this before #59089 and you've added yourself to the maintainers list in both. Can you have one commit for doing that here? like:
|
@worldofpeace thanks for taking the time to review this. Will get the changes in ASAP. |
cd68875
to
8185469
Compare
@worldofpeace I've split the commit into a maintainers and a package commit. |
I actually didn't see the tests being ran in the output.
Looks like we actually just need to use a particular file diff --git a/pkgs/development/python-modules/chevron/default.nix b/pkgs/development/python-modules/chevron/default.nix
index 39fb9d464d0..04de410df68 100644
--- a/pkgs/development/python-modules/chevron/default.nix
+++ b/pkgs/development/python-modules/chevron/default.nix
@@ -1,6 +1,7 @@
{ lib
, buildPythonPackage
, fetchFromGitHub
+, python
}:
buildPythonPackage rec {
@@ -15,6 +16,10 @@ buildPythonPackage rec {
sha256 = "0l1ik8dvi6bgyb3ym0w4ii9dh25nzy0x4yawf4zbcyvvcb6af470";
};
+ checkPhase = ''
+ ${python.interpreter} test_spec.py
+ '';
+
meta = with lib; {
homepage = https://github.com/noahmorrison/chevron;
description = "A python implementation of the mustache templating language";
|
The tests did run for me as is:
|
They succeed for me for |
A Python implementation of mustache
8185469
to
7033cf0
Compare
Added the |
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.
Built every package for every interpreter version.
A Python implementation of mustache
This is being packaged as part of the effort to update
aws-sam-cli
to the latest versionMotivation for this change
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)