Iota2 project pylint compliance.
Several times requests concerning the quality of syntax were made and often partially completed (#456 (closed), #576). Since the pre-commit hook and its configuration file .pre-commit-config.yaml allow pylint recommendations, the purpose of this issue is to ask developers to take some of the pylint messages and fix them.
Here is a table which summarize every pylint messages and their occurrence. Generated thanks to pylint_report.txt
pylint message | occurence | done : |
---|---|---|
R0914, too-many-locals | 135 | |
R0913, too-many-arguments | 29 | |
R0902, too-many-instance-attributes | 50 | |
W1514, unspecified-encoding | 28 | |
R0915, too-many-statements | 43 | |
W0511, fixme | 27 | |
R0912, too-many-branches | 34 | |
C0116, missing-function-docstring | 24 | |
W0613, unused-argument | 26 | |
R1732, consider-using-with | 26 | |
E1101, no-member | 23 | |
C0302, too-many-lines | 21 | |
W0703, broad-except | 20 | |
W0201, attribute-defined-outside-init | 1 | |
R0903, too-few-public-methods | 13 | |
C0415, import-outside-toplevel | 11 | |
C0115, missing-class-docstring | 10 | |
W0707, raise-missing-from | 8 | |
E0611, no-name-in-module | 8 | |
R1728, consider-using-generator | 6 | |
C0103, invalid-name | 4 | |
R0904, too-many-public-methods | 5 | |
W0612, unused-variable | 3 | |
C0113, unneeded-not | 2 | |
W1404, implicit-str-concat | 2 | |
C0200, consider-using-enumerate | 2 | |
W0621, redefined-outer-name | 2 | |
C0301, line-too-long | 2 | |
W3101, missing-timeout | 2 | |
W0107, unnecessary-pass | 2 | |
W0212, protected-access | 1 | |
E0601, used-before-assignment | 1 | |
W0611, unused-import | 1 | |
W0105, pointless-string-statement | 1 | |
W0102, dangerous-default-value | 1 | |
C0206, consider-using-dict-items | 1 | |
E1123, unexpected-keyword-arg | 1 | |
E1137, unsupported-assignment-operation | 1 | |
C0114, missing-module-docstring | 1 | |
E0011, unrecognized-inline-option | 1 |
[UPDATE] : the table has been updated after the removal of
R0913, # too many args
W1203, # logging fstring
C0415, # import outside toplevel
R0913, # too many args
R0902, # too-many-instance-attributes
R0914, # too-many-locals
R0912, # too many branches (to be removed )
R0915, # too many statements (to be removed)
W0511, # TODO (to be removed)
C0302 # too many lines in module
in our .pylintrc file which disable some pylint messages.
developers will have to work together to ensure that they do not assign the same tasks simultaneously. Once a task is done, please change the corresponding