-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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.scipy: use pytestCheckHook && python3.pkgs.numpy: small cleanup #255607
Conversation
a2510ea
to
970167f
Compare
970167f
to
7230254
Compare
I built scipy locally, and tests seem to take long, but I'm not sure. Having another eye on this would be nice. |
7230254
to
e0d075c
Compare
@@ -102,10 +103,14 @@ in buildPythonPackage rec { | |||
|
|||
nativeCheckInputs = [ | |||
pytestCheckHook | |||
# Parallelises tests a bit |
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.
# Parallelises tests a bit |
Related to: NixOS#255262 and a follow-up to NixOS#251923 : - The NOSE_EXCLUDE environment variable was removed due to being unneeded, as upstream uses pytest for a long time now, and that disabled test was removed since. - Support parallel testing with pytest-xdist
e0d075c
to
4936fd3
Compare
Related to: NixOS#255262 . This allows Using the hook's builtin support for of `disabledTests` and `disabledTestPaths`.
4936fd3
to
f2eddda
Compare
I'm closing, as this seems fixed scipy's 1.12.0rc2 . I'll open a new PR with other updates as well when the time comes. |
Related to: #255262 .
Haven't tested this change yet., will undraft this when it is tested.
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)