Updating lint rules to account for new generated files.

googleapis · Jan 7, 2016 · 7a40943 · 7a40943
1 parent f4b1a3c
commit 7a40943
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/scripts/pylintrc_default b/scripts/pylintrc_default
@@ -27,10 +27,8 @@
 # DEFAULT:  ignore=CVS
 # NOTE: This path must be relative due to the use of
 #       os.walk in astroid.modutils.get_module_files.
-# RATIONALE:
-#   _datastore_v1_pb2.py:   protobuf-generated code.
-ignore =
-    _datastore_v1_pb2.py
+# RATIONALE: No files to ignore.
+ignore=
 
 # Pickle collected data for later comparisons.
 # DEFAULT:  persistent=yes

diff --git a/scripts/run_pylint.py b/scripts/run_pylint.py
@@ -29,11 +29,11 @@
 
 
 IGNORED_DIRECTORIES = [
-    'gcloud/bigtable/_generated',
+    os.path.join('gcloud', 'bigtable', '_generated'),
+    os.path.join('gcloud', 'datastore', '_generated'),
 ]
 IGNORED_FILES = [
-    'gcloud/datastore/_datastore_v1_pb2.py',
-    'docs/conf.py',
+    os.path.join('docs', 'conf.py'),
     'setup.py',
 ]
 SCRIPTS_DIR = os.path.abspath(os.path.dirname(__file__))

diff --git a/tox.ini b/tox.ini
@@ -61,7 +61,7 @@ deps = {[testenv:docs]deps}
 passenv = {[testenv:docs]passenv}
 
 [pep8]
-exclude = gcloud/datastore/_datastore_v1_pb2.py,gcloud/bigtable/_generated/*,docs/conf.py
+exclude = docs/conf.py,gcloud/bigtable/_generated/*,gcloud/datastore/_generated/*
 verbose = 1
 
 [testenv:lint]