flake8-eradicate | Flake8 plugin to find commented out or dead code | Code Analyzer library
kandi X-RAY | flake8-eradicate Summary
kandi X-RAY | flake8-eradicate Summary
Flake8 plugin to find commented out or dead code
Support
Quality
Security
License
Reuse
Top functions reviewed by kandi - BETA
Currently covering the most popular Java, JavaScript and Python libraries. See a Sample of flake8-eradicate
flake8-eradicate Key Features
flake8-eradicate Examples and Code Snippets
Community Discussions
Trending Discussions on flake8-eradicate
QUESTION
I'm trying to run flakehell as pre-commit hook.
my .pre-commit-config.yaml:
...ANSWER
Answered 2021-Jan-11 at 20:19your configuration is incorrect, you haven't limited the files that are passed to your hook with either files
or types
so it is defaulting to all files in your repository. presumably you have some binary file which is being passed to flakehell
I also notice that your configuration passes both a path and has pass_filenames: true
(pass_filenames: true
is the default so you shouldn't use that)
you either want to list paths in args (not recommended since you always lint more than what you're changing) or you want to filter the filenames properly
additionally, verbose: true
is not intended for use outside of debugging as it adds warning noise to the output
additionally, you're not managing the installation of flakehell through pre-commit which will add additional burden to your contributors to try and set up whatever development environment locally, most of the point of pre-commit is that it manages installing your tools so your contributors don't have to jump through hoops to have the correct formatting / linting setup (eliminating a whole class of "it works on my machine" problems)
additionally, it looks like flakehell has direct support for pre-commit, so you don't need to use the repo: local
escape hatch as you're doing
putting all of that together, you probably want something like this:
Community Discussions, Code Snippets contain sources that include Stack Exchange Network
Vulnerabilities
No vulnerabilities reported
Install flake8-eradicate
Support
Reuse Trending Solutions
Find, review, and download reusable Libraries, Code Snippets, Cloud APIs from over 650 million Knowledge Items
Find more librariesStay Updated
Subscribe to our newsletter for trending solutions and developer bootcamps
Share this Page