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
doc/python: add pytestCheckHook section #95700
Conversation
dc69448
to
45df001
Compare
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.
Looks pretty good to me! This would've been very helpful when I was first starting to learn nix. Thanks for taking the time to write this up. Few spelling/wording changes suggested.
45df001
to
233dc9c
Compare
@drewrisinger thanks for the review, applied/adapted most of the suggestions |
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. Only last suggestion would be a link to the pytest documentation for more details: https://docs.pytest.org/en/stable/usage.html
maybe once the ecosystem stabilizes around pytest again, see #95609 (comment) and #95609 (comment) but I'm just afraid that they will be fed information which is against the common practices in nixpkgs (currently) |
Motivation for this change
noticed in #95605 that we didn't have a good example of how to use
pytestCheckHook
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)