[tor-commits] [arm/master] Remove PEP8 blacklisting

atagar at torproject.org atagar at torproject.org
Sun Jan 19 00:34:25 UTC 2014


commit 6eaf35f08205c9717ebbe37bbf46d7e43abf0935
Author: Damian Johnson <atagar at torproject.org>
Date:   Sat Jan 18 16:34:33 2014 -0800

    Remove PEP8 blacklisting
    
    Now that arm is PEP8 compliant we don't need the ability to blacklist files
    from the checks.
---
 run_tests.py |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/run_tests.py b/run_tests.py
index dd7ff98..2c5b380 100755
--- a/run_tests.py
+++ b/run_tests.py
@@ -18,7 +18,6 @@ from arm.util import load_settings
 
 CONFIG = stem.util.conf.config_dict("test", {
   "pep8.ignore": [],
-  "pep8.blacklist": [],
   "pyflakes.ignore": [],
 })
 
@@ -144,12 +143,9 @@ def get_stylistic_issues(paths):
           issues.setdefault(self.filename, []).append((offset + line_number, "%s %s" % (code, text)))
 
     style_checker = pep8.StyleGuide(ignore = CONFIG["pep8.ignore"], reporter = StyleReport)
-    style_checker.check_files(filter(lambda path: path not in CONFIG["pep8.blacklist"], _python_files(paths)))
+    style_checker.check_files(_python_files(paths))
 
   for path in _python_files(paths):
-    if path in CONFIG["pep8.blacklist"]:
-      continue
-
     with open(path) as f:
       file_contents = f.read()
 



More information about the tor-commits mailing list