This repository has been archived by the owner on May 16, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 267
Drop flake8? #292
Labels
Comments
If pylint can do everything, why not? |
+1 |
1 similar comment
+1 |
@dreispt |
@moylop260 I'm sorry but no - in the little time I will have I will give priority to a few new modules I'm in need. |
I got it |
Closed
As mentioned in #286, this would also allow to ignore non installable modules. |
I was talking with @moylop260 and even pylint team delegate some work to Flake8. This will be simplified on the move on #500 and future work, can I close this issue? |
The pylint’s maintainer recommend us use both tools: |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
MQT currently uses two tools for linting, flake8 and pylint, and I see people puzzled at why the complexity.
Indeed this is so for historical reasons (pylint was added only later in the game), but at the moment I feel that pylint is capable of doing everything and more that flake8 can.
So I propose to move the existing flake8 checks to pylint checks, and remove the flake8 dependency from MQT.
The text was updated successfully, but these errors were encountered: