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.py_stringmatching: init at 0.4.0 #35471
Conversation
sha256 = "0rjsx7iipn6svki21lmsza7b0dz9vkgmix696zryiv7gkhblqyb4"; | ||
}; | ||
|
||
# No tests in archive |
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.
Wrong! Just look in py_stringmatching/tests/
.
Please enable tests and use nosetests
in the checkPhase
.
propagatedBuildInputs = [ numpy six ]; | ||
|
||
meta = with lib; { | ||
description = "This project seeks to build a Python software package that consists of a comprehensive and scalable set of string tokenizers (such as alphabetical tokenizers, whitespace tokenizers) and string similarity measures (such as edit distance, Jaccard, TF/IDF)."; |
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.
Too long. This could be a longDescription
. Also please split it into multiple lines.
@@ -0,0 +1,29 @@ | |||
|
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.
unnecessary whitespace
@dotlambda should be alright now! |
@GrahamcOfBorg build python2.pkgs.py_stringmatching python3.pkgs.py_stringmatching |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
propagatedBuildInputs = [ numpy six ]; | ||
|
||
meta = with lib; { | ||
description = "A Python string matching library including string tokenizers and string similarity measures."; |
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.
No period. See https://nixos.org/nixpkgs/manual/#sec-standard-meta-attributes.
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.
Also, you can keep the previous description
as longDescription
if you want.
@dotlambda - I applied the requested changes. |
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)