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
python38Packages.percol: python3 support #115729
Conversation
@SuperSandro2000 changes made as requested, please review |
Hi. Can this be merged? Are there any further changes necessary? |
This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 4 packages built:
The following issues got detected with the above build packages.
python38Packages.cmigemo:
warning: python-include-tests See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/python-include-tests.md |
@SuperSandro2000 I've added pytest for cmigemo. percol didn't have tests, so I added pythonImportsCheck. |
pythonImportsCheck can be added to all python packages even if running tests. They make sure the python package can be imported which is important that all required packages are there even without checkInputs. Also sorry for letting you wait. There are just to many PRs and to little time. |
Yes, the number of PRs is terrifying XD Should I add pythonImportsCheck to cmigemo? Looks like pytest already tests this |
It wouldn't hurt because checkInputs has some extra packages most likely which are not available in the final environment. |
Motivation for this change
The percol package is currently marked as broken on python3
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)