lint: clean up old statements
We can delete a number of disable statements that are no longer relevant,
replace the W/E/F constants with their human readable names, and scope a
few to where they're needed instead of disabling in the entire module.
BUG=None
TEST=`cros lint` is unchanged
Change-Id: Ic7ac1ab410917b075212b38412f8a1e381b8b915
Reviewed-on: https://chromium-review.googlesource.com/1135635
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>
diff --git a/scripts/cbuildbot.py b/scripts/cbuildbot.py
index df496ad..ecd2f42 100644
--- a/scripts/cbuildbot.py
+++ b/scripts/cbuildbot.py
@@ -569,7 +569,7 @@
# Debug options
#
# Temporary hack; in place till --dry-run replaces --debug.
- # pylint: disable=W0212
+ # pylint: disable=protected-access
group = parser.debug_group
debug = [x for x in group.option_list if x._long_opts == ['--debug']][0]
debug.help += ' Currently functions as --dry-run in addition.'
@@ -677,7 +677,7 @@
'running the %s config', constants.BRANCH_UTIL_CONFIG)
-# pylint: disable=W0613
+# pylint: disable=unused-argument
def _PostParseCheck(parser, options, site_config):
"""Perform some usage validation after we've parsed the arguments