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.eradicate: init at 0.2.1 #47894
Conversation
I wanted to run |
You mean the script is in the tarball, but not installed into |
oops so I was mistaken. I thought for some reason I was fetching source from github where the test file lies, but its actually coming from pypi where its never packaged. So I think this is about as good as can be and is ready @FRidh |
}; | ||
|
||
meta = with lib; { | ||
description = "eradicate removes commented-out code from Python files."; |
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 capitalize the first letter and remove the last .
.
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.
Good one, I missed it.
description = "eradicate removes commented-out code from Python files."; | ||
homepage = https://github.com/myint/eradicate; | ||
license = [ licenses.mit ]; | ||
|
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.
(minor) useless blank line
This package has been merged (80651cf) |
oops didn't see the comments by @nlewo sorry. |
Motivation for this change
This is a new dependency of pylama, which I intend to PR with the upgrade.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)