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
pythonPackages.pytest-flake8: 0.9.1 -> 1.0.0 #37971
Conversation
4c76695
to
5f93c20
Compare
@@ -3,7 +3,7 @@ | |||
buildPythonPackage rec { | |||
name = "${pname}-${version}"; |
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.
Please drop this line.
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.
Ok! 👍
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
@@ -3,7 +3,7 @@ | |||
buildPythonPackage rec { | |||
name = "${pname}-${version}"; | |||
pname = "pytest-flake8"; | |||
version = "0.9.1"; | |||
version = "1.0.0"; | |||
|
|||
# although pytest is a runtime dependency, do not add it as | |||
# propagatedBuildInputs in order to allow packages depend on another version |
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 believe this comment should be removed and pytest
moved to propagatedBuildInputs
, because applications can use their own pytest
version using packageOverrides
. Shouldn't it, @FRidh?
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.
In case of pytest
it is not just for applications, but also for libraries. Certain libraries may request a certain pytest
version.
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.
So I'll keep it as it is or change it somehow?
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.
It can stay as it is.
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.
Ok, thanks! Then I think this should be ready to merge.
5f93c20
to
aef1900
Compare
Motivation for this change
Upgrade pytest-flake8.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)