summaryrefslogtreecommitdiff
path: root/Meta/lint-python.sh
diff options
context:
space:
mode:
authorAndrew Kaster <andrewdkaster@gmail.com>2021-01-10 00:20:21 -0700
committerAndreas Kling <kling@serenityos.org>2021-01-10 16:39:21 +0100
commit9e5aa6f794ef9ab5828d5874c2e40de9a4fa8349 (patch)
treebb2918d12e19ff08f123e286f5b90d7c9bbe8e77 /Meta/lint-python.sh
parent190e0e155130e09dc9c9b7a0989cdb0ef8c55a7d (diff)
downloadserenity-9e5aa6f794ef9ab5828d5874c2e40de9a4fa8349.zip
Meta: Only complain about linter tools if relevant files have changed
Diffstat (limited to 'Meta/lint-python.sh')
-rwxr-xr-xMeta/lint-python.sh10
1 files changed, 5 insertions, 5 deletions
diff --git a/Meta/lint-python.sh b/Meta/lint-python.sh
index 790f2035e1..494c2a7bc2 100755
--- a/Meta/lint-python.sh
+++ b/Meta/lint-python.sh
@@ -5,11 +5,6 @@ set -e
script_path=$(cd -P -- "$(dirname -- "$0")" && pwd -P)
cd "${script_path}/.." || exit 1
-if ! command -v flake8 >/dev/null 2>&1 ; then
- echo "flake8 is not available. Either skip this script, or install flake8."
- exit 1
-fi
-
if [ "$#" -eq "0" ]; then
mapfile -t files < <(
git ls-files '*.py'
@@ -24,6 +19,11 @@ else
fi
if (( ${#files[@]} )); then
+ if ! command -v flake8 >/dev/null 2>&1 ; then
+ echo "flake8 is not available, but python files need linting! Either skip this script, or install flake8."
+ exit 1
+ fi
+
flake8 "${files[@]}" --max-line-length=120
else
echo "No py files to check."