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
python3.pkgs.pyexcel: init at 0.6.4 #96483
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.
for packages that can't run tests, do you mind adding pythonImportsCheck
?
If tests are not available, then please use pythonImportsCheck
to import the most important modules. This isn't as good as unit tests, but will usually give a good indication of run-time errors. Please see pythonImportsCheck documentation for more information.
Done, thanks. |
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.
LGTM
Result of nixpkgs-review pr 96483 1
13 packages built:
- python27Packages.lml
- python27Packages.pyexcel-io
- python27Packages.pyexcel-xls
- python37Packages.lml
- python37Packages.pyexcel
- python37Packages.pyexcel-io
- python37Packages.pyexcel-ods
- python37Packages.pyexcel-xls
- python38Packages.lml
- python38Packages.pyexcel
- python38Packages.pyexcel-io
- python38Packages.pyexcel-ods
- python38Packages.pyexcel-xls
Motivation for this change
Just needed this again so I finally packaged it.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
nix-shell -I nixpkgs=$HOME/Projects/nixpkgs -p 'python3.withPackages(ps: with ps; [ pyexcel pyexcel-ods ])' --run 'python3 -c "import pyexcel; print(pyexcel.get_records(file_name=\"data.ods\", sheet_name=\"INFO\"))"'
nix path-info -S
before and after)