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
watson: 1.5.2 -> 1.7.0 #63972
watson: 1.5.2 -> 1.7.0 #63972
Conversation
dbec15d
to
e281f7a
Compare
e281f7a
to
550a265
Compare
84f6b7c
to
f7710fd
Compare
I assume that the merge will be blocked until those are fixed, then. Just to ensure that I've understood it correctly: The package is located in |
I can bypass that locally by disabling the
Yes that's correct. However I usually advise people to normalize |
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.
Can you please drop doCheck = true;
(some reason can't suggest that on GitHub) and use the new pytest
command in the checkPhase
?
Other than that LGTM.
Motivation for this change
Update to the latest version on PyPI.
Current nixos-version:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review 63972"
./result/bin/
)nix path-info -S
before and after)