From cc49048a58473a4ca7889f1389f0362149822df7 Mon Sep 17 00:00:00 2001 From: Jan Richter Date: Wed, 26 Jun 2024 10:58:19 +0200 Subject: [PATCH] pylint-3 removal Avocado is using pylint-3 instead of default pylint. This is a leftover from the time when avocado supported python2. Unfortunately, the pylint-3 has slightly different behaviour than pylint, therefore we need to update it to have more accurate testing. Lets use python3 -m pylint Signed-off-by: Jan Richter --- selftests/lint.sh | 4 +--- selftests/spell.sh | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/selftests/lint.sh b/selftests/lint.sh index a73635b6e7..561b669440 100755 --- a/selftests/lint.sh +++ b/selftests/lint.sh @@ -1,9 +1,7 @@ #!/bin/sh -e echo "** Running lint..." -PYLINT=$(which pylint-3 2>/dev/null || which pylint) - # Those are files from our main packages, we should follow the .pylintrc file with all # enabled by default. Some are disabled, we are working to reduce this list. FILES=$(git ls-files '*.py') -${PYLINT} ${PYLINT_OPTIONS} ${FILES} +python3 -m pylint ${PYLINT_OPTIONS} ${FILES} diff --git a/selftests/spell.sh b/selftests/spell.sh index f013597d19..862468a496 100755 --- a/selftests/spell.sh +++ b/selftests/spell.sh @@ -1,6 +1,4 @@ #!/bin/sh -e echo "** Running spell check..." -PYLINT=$(which pylint-3 2>/dev/null || which pylint) - -${PYLINT} -j 1 --errors-only --disable=all --enable=spelling --spelling-dict=en_US --spelling-private-dict-file=spell.ignore * +python3 -m pylint -j 1 --disable=all --enable=spelling --spelling-dict=en_US --spelling-private-dict-file=spell.ignore *