History | View | Annotate | Download (2.8 kB)
Reduce pylint maximum file length to 4500
The longest Python files we still have are around 4200 lines long. Inorder to prevent future growth, limit the maximum file length (checkedby pylint) to 4500 lines.
Signed-off-by: Thomas Thrainer <thomasth@google.com>...
Rename some constants and relax pylint rules
For the type system, we want a slightly relaxed rule for constantnaming, so we update the pylint rule. But the old _TPInt and _TNEStringwere not clear enough, so we expand them.
Signed-off-by: Iustin Pop <iustin@google.com>...
Improve pylintrc for pylint 0.21+
While we'll need to update the source files too, at least this changemakes pylint 0.21 not fail on the current source tree.
Signed-off-by: Iustin Pop <iustin@google.com>Reviewed-by: Guido Trotter <ultrotter@google.com>
pylint: Temporarily disable W0201
There seems to be a bug in pylint relating to W0201 (“Attribute '%r'defined outside init”) being re-enabled by simple comments and notbeing able to be disabled again.
Until that is fixed, we disable this warning....
Disable R0922 in pylint
R0922, “Abstract class is only referenced 1 times”, cannot be disabledin the source code, and thuslib/http/auth.py:HttpServerRequestAuthentication gives this warning(it's actually also referenced from the tests, but we do not include...
pylint: disable the similarities checker
This is a very slow checker, estimated to be O(n²) by its author; sousing it all the time is not good. It can be re-enabled on the commandline via “--disable-checker=”.
Disable TODO warnings in pylint
We don't get any useful out of this - a git grep is as effective, andthese only pollute the pylint output.
Signed-off-by: Iustin Pop <iustin@google.com>Reviewed-by: Olivier Tharan <olive@google.com>
Merge branch 'devel-2.0' into devel-2.1
Update pylintrc
Since the current pylintrc is in now way good for our style, we updateit:
- remove docstring checks, since we have too many cases where a docstring doesn't make sense- relax naming rules to cover more alternatives (but not break the naming rules)...
Disallow "xrange" function
Signed-off-by: Michael Hanselmann <hansmi@google.com>Reviewed-by: Iustin Pop <iustin@google.com>
Add custom pylintrc