pylint: Remove R0401 and R0904 disables
- R0401: cyclic-import
- R0904: max-public-methods to 20
- https://pylint.readthedocs.io/en/latest/technical_reference/features.html
Edited by Moul
Merge request reports
Activity
changed milestone to %1.1.0
added Dev Env label
requested review from @vtexier
assigned to @moul
I completely agree with the cyclic import R0401. Because a Python program with cyclic import will break.
I do not like maximum or minimum numbers of functions, methods, attributes. As every limit on a creative work, numbers are completely arbitrary. Why 20? Why not 21 or 15 or 25? I have already meet this kind of error in one of my project, that would have lead to useless fragmentation. Let's do it, but we will remove R0904 as soon as it stands on our way.
added 3 commits
-
540e5a8f...9e0c19cf - 2 commits from branch
main
- 3389e8d7 - [lint] Remove R0401 and R0904 disables
-
540e5a8f...9e0c19cf - 2 commits from branch
enabled an automatic merge when the pipeline for 3389e8d7 succeeds
Please register or sign in to reply