[virt-tools-list] [PATCH 01/17] pylint: Don't exclude progress.py

Radostin Stoyanov rstoyanov1 at gmail.com
Thu Oct 25 12:37:42 UTC 2018


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 <rstoyanov1 at gmail.com>
---
 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]
 
-- 
2.17.2




More information about the virt-tools-list mailing list