commit d35d563a73b0cfbb26b3b3b4e21a735a29ed3d5c
author Ryo Hashimoto <hashimoto@google.com> Wed Aug 01 15:53:42 2018 +0900
committer chrome-bot <chrome-bot@chromium.org> Wed Aug 01 14:52:27 2018 -0700
tree d5ea1e728874aa33815dcbfffa7940cca3ee788b
parent 598490e12b0dcb8c822dfddf1f7af2c9be86fa65

utils: Fix run_pylint.py

run_pylint.py is broken because of pylint upgrade in depot_tools
(CL:1145485)

BUG=chromium:863669
TEST=Create an autotest CL and repo upload

Change-Id: I7a0407a862f0b2bcc41b50d2857c4c6184c3d540
Reviewed-on: https://chromium-review.googlesource.com/1158101
Commit-Ready: Ryo Hashimoto <hashimoto@chromium.org>
Tested-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

utils/run_pylint.py

diff --git a/utils/run_pylint.py b/utils/run_pylint.py
index bb3f4cc..61eda1b 100755
--- a/utils/run_pylint.py
+++ b/utils/run_pylint.py
@@ -121,10 +121,10 @@
 
 class CustomImportsChecker(imports.ImportsChecker):
     """Modifies stock imports checker to suit autotest."""
-    def visit_from(self, node):
+    def visit_importfrom(self, node):
         """Patches modnames so pylints understands autotest_lib."""
         node.modname = patch_modname(node.modname)
-        return super(CustomImportsChecker, self).visit_from(node)
+        return super(CustomImportsChecker, self).visit_importfrom(node)
 
 
 class CustomVariablesChecker(variables.VariablesChecker):
@@ -146,10 +146,10 @@
         patch_consumed_list(scoped_names[0],scoped_names[1])
         self._to_consume.append(scoped_names)
 
-    def visit_from(self, node):
+    def visit_importfrom(self, node):
         """Patches modnames so pylints understands autotest_lib."""
         node.modname = patch_modname(node.modname)
-        return super(CustomVariablesChecker, self).visit_from(node)
+        return super(CustomVariablesChecker, self).visit_importfrom(node)
 
 
 class CustomDocStringChecker(base.DocStringChecker):
@@ -164,7 +164,7 @@
         pass
 
 
-    def visit_function(self, node):
+    def visit_functiondef(self, node):
         """
         Don't request docstrings for commonly overridden autotest functions.
 
@@ -184,7 +184,7 @@
         if _is_test_case_method(node):
             return
 
-        super(CustomDocStringChecker, self).visit_function(node)
+        super(CustomDocStringChecker, self).visit_functiondef(node)
 
 
     @staticmethod