Suppress new pylint 1.6 opinions for *all* files, not just tests.

tseaver · tseaver · commit 06800cf7b6ed · 2016-07-11T17:06:22.000-04:00
See googleapis#1968 for eventual reversal. This change reverts googleapis#1957 in favor of a more global suppression.
diff --git a/scripts/pylintrc_default b/scripts/pylintrc_default
@@ -90,6 +90,13 @@ load-plugins=pylint.extensions.check_docs
 # - no-name-in-module: Error gives a lot of false positives for names which
 #                      are defined dynamically. Also, any truly missing names
 #                      will be detected by our 100% code coverage.
+#
+# New opinions in pylint 1.6, enforcing PEP 257.  #1968 for eventual fixes
+# - catching-non-exception
+# - missing-raises-doc
+# - missing-returns-doc
+# - redundant-returns-doc
+# - ungrouped-imports
 disable =
     maybe-no-member,
     no-member,
@@ -99,6 +106,11 @@ disable =
     redefined-variable-type,
     wrong-import-position,
     no-name-in-module,
+    catching-non-exception,
+    missing-raises-doc,
+    missing-returns-doc,
+    redundant-returns-doc,
+    ungrouped-imports
 
 
 [REPORTS]
diff --git a/scripts/run_pylint.py b/scripts/run_pylint.py
@@ -51,8 +51,6 @@
     'import-error',
     'invalid-name',
     'missing-docstring',
-    'missing-raises-doc',
-    'missing-returns-doc',
     'no-init',
     'no-self-use',
     'superfluous-parens',
