From 6066772ca8e8290b0f6eb240b06b8e19ca4844a0 Mon Sep 17 00:00:00 2001 From: Caleb Connolly Date: Mon, 21 Oct 2024 22:43:06 +0200 Subject: [PATCH] CI: ruff: run ALLL the checks before failing (MR 2448) So we don't go through the pain of pushing 3 times to get 3 different ruff failures in CI. Signed-off-by: Caleb Connolly --- .ci/ruff.sh | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/.ci/ruff.sh b/.ci/ruff.sh index 05d5289d..e77864d8 100755 --- a/.ci/ruff.sh +++ b/.ci/ruff.sh @@ -8,15 +8,19 @@ if [ "$(id -u)" = 0 ]; then exec su "${TESTUSER:-build}" -c "sh -e $0" fi +DID_FAIL=0 + set -x # __init__.py with additional ignore: # F401: imported, but not used # shellcheck disable=SC2046 -ruff check --ignore "F401" $(find . -not -path '*/venv/*' -name '__init__.py') +ruff check --ignore "F401" $(find . -not -path '*/venv/*' -name '__init__.py') || DID_FAIL=1 # Check all other files -ruff check --exclude=__init__.py . +ruff check --exclude=__init__.py . || DID_FAIL=1 # Check formatting -ruff format --diff +ruff format --diff || DID_FAIL=1 + +exit $DID_FAIL