Flake8 support
See original GitHub issueIs your feature request related to a problem? Please describe. I want to use flake8 instead of pylint
Describe the solution you’d like Allow choosing between flake8 and pylint
Describe alternatives you’ve considered
Before I opened this issue, I wanted to make a proof of concept to see how easy/difficult would it be to run flake8 using this plugin. I wrote a simple flake report formatter to produce json output that match the output you expect from pylint. Then I changed pylint executable to run flake8 instead after correcting the parameters (i.e --format=json
instead of -f json
). The plugin worked flawlessly.
Additional context Flake allows passing its output format as parameters like that
flake8 --format="%(path)s:%(row)d:%(col)d:%(code)s:%(text)s"
Then you get output like that
orders/models.py:105:9:F841:local variable 'c' is assigned to but never used
orders/models.py:105:9:WPS111:Found too short name: c < 2
orders/models.py:107:12:F821:undefined name 'z'
In python (I don’t know Java very well), It can be parsed in one line
path, line, column, message_id, message = violation_line.split(":", maxsplit=4)
Also --rc-file
parameter is to be replaced with --config
parameter
Issue Analytics
- State:
- Created 3 years ago
- Comments:23 (5 by maintainers)
Top GitHub Comments
Nice! I think you built it! Try with the runIde task (not 100% sure about the task name, don’t have the PC to check right now).
I’m also going to bed, I’ll check your progress tomorrow 😃