There are more pylint checks which can be enabled in webkitpy/pylintrc which could potentially be helpful --; some pylint checks that are currently disabled that we may want to enable include:
# C0103: Invalid name (list of exceptions can be set)
# C0301: Line too long (max length can be set)
# C0302: Too many lines in module (max length can be set)
# I0010: Unable to consider inline option (possibly improperly formatted pylint: disable pragma)
# R0801: Similar lines in other files
# W0212: Access to a protected member (with underscore) of a client class
# W0402: Uses of a deprecated module
# W0404: Reimported module (imported multiple times)
# W0603: Using the global statement
# W0703: Broad except statement (except Exception)
# W1201: Logging statements that don't format string format arguments as logging function parameters
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Tools/Scripts/webkitpy/pylintrc&l=67
Additionally, the pylint runner function in webkitpy.style.checkers.python only prints errors, not warnings, so pylint warnings are currently ignored in practice; but as long as all of the warnings are things that we want to pay attention to, then it may be helpful to print warnings too.
Any thoughts (or checks above that aren't worth enabling)?
Comment 1 by bugdroid1@chromium.org
, Mar 30 2016