From 5b1f6ad2b8207d713c44e2f5beb36058ef8d0500 Mon Sep 17 00:00:00 2001 From: Radostin Stoyanov Date: Thu, 25 Oct 2018 13:37:42 +0100 Subject: [PATCH] pylint: Don't exclude progress.py Currently progress.py file was excluded from pylint/pycodestyle check because the code was straight from python-urlgrabber and it was not PEP8 compliant. The following patched resolve the code style issues. Signed-off-by: Radostin Stoyanov --- setup.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/setup.py b/setup.py index 2e1215c0..658e6503 100755 --- a/setup.py +++ b/setup.py @@ -586,16 +586,12 @@ class CheckPylint(distutils.core.Command): "tests"] output_format = sys.stdout.isatty() and "colorized" or "text" - exclude = ["virtinst/progress.py"] print("running pycodestyle") style_guide = pycodestyle.StyleGuide( config_file='tests/pycodestyle.cfg', paths=files ) - style_guide.options.exclude = pycodestyle.normalize_paths( - ','.join(exclude) - ) report = style_guide.check_files() if style_guide.options.count: sys.stderr.write(str(report.total_errors) + '\n') @@ -604,7 +600,7 @@ class CheckPylint(distutils.core.Command): pylint_opts = [ "--rcfile", "tests/pylint.cfg", "--output-format=%s" % output_format, - ] + ["--ignore"] + [os.path.basename(p) for p in exclude] + ] if self.jobs: pylint_opts += ["--jobs=%d" % self.jobs]