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
Add nplusone and pytest-pythonpath Python packages #94054
Conversation
# The tests assume the source code is in an nplusone/ directory. When using | ||
# the Nix sandbox, it will be in a source/ directory instead, making the | ||
# tests fail. | ||
patches = [ ./fix_tests.diff ]; |
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.
I would prefer substituteInPlace over a diff, generally they are less likely to fail to apply when bumping the package.
patches = [ ./fix_tests.diff ]; | |
prePatch = '' | |
substituteInPlace tests/conftest.py \ | |
--replace nplusone/tests/conftest source/tests/conftest | |
''; |
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.
Done, nice suggestion!
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.
To comply with CONTRIBUTING.md please have the commit message name be of the format
<pkg-name>: <subject-line>
for more examples, please look at https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#submitting-changes
in your case, the commit message should be:
pythonPackages.pytest-pythonpath: init at 0.7.3
pythonPackages.nplusone: init at 1.0.0
you can use git rebase -i
to re-order and squash the commits in the correct order
I didn't fully undertand the suggestion about the commit messages. Right now, all commits are using the
Did you mean I should squash 5dfa72af62d and a5ac33d7b13 into the previous commits so we only have the |
Inspired in the suggestions made by @jonringer in NixOS/nixpkgs#94054 (comment)
I meant that the fixup commits should be squashed into their respective package addition commits |
Done, thanks for clarifying! |
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 94054 1
5 packages built:
- python27Packages.pytest-pythonpath
- python37Packages.nplusone
- python37Packages.pytest-pythonpath
- python38Packages.nplusone
- python38Packages.pytest-pythonpath
Motivation for this change
Add the
nplusone
Python package, which is a dependency of Faraday.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)