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
vulture: init at 1.6 #94396
vulture: init at 1.6 #94396
Conversation
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.
Thanks for your contribution! The code looks good.
It seems dependencies are missing:
ERROR: could not install deps [coverage, pytest, pytest-cov]; v = InvocationError(u'/build/vulture-1.6/.tox/py/bin/python -m pip install coverage pytest pytest-cov', 1)
3 packages failed to build:
python27Packages.vulture python37Packages.vulture python38Packages.vulture
https://gist.github.com/davidak/71903b64737e4d30a6e287c754c1e3ac
@davidak thanks for catching that. I hadn't yet tried to build with sandboxing (but now realize that is super important!). I think the issue was trying to run the tests with I found a couple of examples of packages using |
pkgs/top-level/python-packages.nix
Outdated
@@ -7697,6 +7697,7 @@ in { | |||
|
|||
rxv = callPackage ../development/python-modules/rxv { }; | |||
|
|||
vulture = callPackage ../development/python-modules/vulture { }; |
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 try to add this in a semi-sorted fashion. Appending to the end will cause merge conflicts to others who do the same.
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.
Fixed in 0674589.
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.
should be two commits:
maintainers: add mcwitt
pythonPackages.vulture: init at 1.6
mcwitt = { | ||
email = "mcwitt@gmail.com"; | ||
github = "mcwitt"; | ||
githubId = 319411; | ||
name = "Matt Wittmann"; | ||
}; |
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 make this into a separate commit:
maintainers: add mcwitt
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 in d988033.
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 94396 1
Motivation for this change
Adds the vulture tool for detecting dead code in Python.
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)