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.pycoin: init at 0.90.20200322 #85994
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.
Formatting looks pretty good.
Few comments on reducing extra patch files/verbosity of test section.
@GrahamcOfBorg build python27Packages.pycoin python37Packages.pycoin python38Packages.pycoin
pkgs/development/python-modules/pycoin/hardcode-gnupg-path.patch
Outdated
Show resolved
Hide resolved
@GrahamcOfBorg build python27Packages.pycoin python37Packages.pycoin python38Packages.pycoin |
05b6579
to
4c53501
Compare
Hello @drewrisinger |
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.
Diff LGTM minus comments.
Haven't built.
@GrahamcOfBorg build python27packages.pycoin python37Packages.pycoin python38Packages.pycoin |
4c53501
to
059344c
Compare
@GrahamcOfBorg build python27Packages.pycoin python37Packages.pycoin python38Packages.pycoin |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
diff LGTM
commits LGTM
has tests 👍
https://github.com/NixOS/nixpkgs/pull/85994
3 packages built:
python27Packages.pycoin python37Packages.pycoin python38Packages.pycoin
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)