commit 27e21b72d0d7861e2a74ec095f39267e62c46422
author Mike Frysinger <vapier@chromium.org> Thu Jul 12 14:20:21 2018 -0400
committer chrome-bot <chrome-bot@chromium.org> Fri Jul 13 17:29:15 2018 -0700
tree a70bb3f471e327beea0d9995742a4deef2453587
parent deed023a0f61dde315e6e13a7fb465e911c9a9bb

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>

scripts/gerrit.py

diff --git a/scripts/gerrit.py b/scripts/gerrit.py
index 5d9812e..2b3bb9c 100644
--- a/scripts/gerrit.py
+++ b/scripts/gerrit.py
@@ -587,7 +587,7 @@
   opts.gerrit = {}
   opts.Freeze()
 
-  # pylint: disable=W0603
+  # pylint: disable=global-statement
   global COLOR
   COLOR = terminal.Color(enabled=opts.color)