python: Fixing some python2-to-python3 migrations that I missed in the last PR.

This commit is contained in:
Christian Parpart 2020-01-31 14:26:55 +01:00 committed by chriseth
parent 6ab385d34a
commit 07a04bed44

View File

@ -1,14 +1,14 @@
#! /usr/bin/env python3
"""
Performs pylint on all python files in the project repo's test directory recursively.
Performs pylint on all python files in the project repo's {test,script,docs} directory recursively.
This script is meant to be run from the CI but can also be easily in local dev environment,
where you can optionally pass `-d` as command line argument to let this script abort on first error.
"""
from os import path, walk, system
from sys import argv, exit as brexit
from sys import argv, exit as exitwith
PROJECT_ROOT = path.dirname(path.realpath(__file__))
PYLINT_RCFILE = "{}/pylintrc".format(PROJECT_ROOT)
@ -36,19 +36,20 @@ def pylint_all_filenames(dev_mode, rootdirs):
if exit_code != 0:
if dev_mode:
return 1, checked_count
else:
failed.append(filename)
failed.append(filename)
return len(failed), len(filenames)
def main():
"""" Collects all python script root dirs and runs pylint on them. """
""" Collects all python script root dirs and runs pylint on them. You can optionally
pass `-d` as command line argument to let this script abort on first error. """
dev_mode = len(argv) == 2 and argv[1] == "-d"
failed_count, total_count = pylint_all_filenames(dev_mode, [
path.abspath(path.dirname(__file__) + "/../docs"),
path.abspath(path.dirname(__file__) + "/../scripts"),
path.abspath(path.dirname(__file__) + "/../test")])
if failed_count != 0:
brexit("pylint failed on {}/{} files.".format(failed_count, total_count))
exitwith("pylint failed on {}/{} files.".format(failed_count, total_count))
else:
print("Successfully tested {} files.".format(total_count))