diff options
author | Andrew Kaster <andrewdkaster@gmail.com> | 2021-01-10 00:20:21 -0700 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-01-10 16:39:21 +0100 |
commit | 9e5aa6f794ef9ab5828d5874c2e40de9a4fa8349 (patch) | |
tree | bb2918d12e19ff08f123e286f5b90d7c9bbe8e77 /Meta/lint-python.sh | |
parent | 190e0e155130e09dc9c9b7a0989cdb0ef8c55a7d (diff) | |
download | serenity-9e5aa6f794ef9ab5828d5874c2e40de9a4fa8349.zip |
Meta: Only complain about linter tools if relevant files have changed
Diffstat (limited to 'Meta/lint-python.sh')
-rwxr-xr-x | Meta/lint-python.sh | 10 |
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." |