From 5fdb54cb88baf9ffad8dc134f7925fe85d60fb44 Mon Sep 17 00:00:00 2001 From: MrFastwind Date: Fri, 24 Jul 2026 19:04:25 +0200 Subject: [PATCH] ci: fix pipefail killing lint step before grep check, uncomment artifact upload - Add '|| true' to SQFVM pipeline so pipefail doesn't kill the step before we can grep for [ERR] lines - Uncomment lint log artifact upload step - description.ext false positive filtered by grep -v --- .gitea/workflows/build.yml | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/.gitea/workflows/build.yml b/.gitea/workflows/build.yml index f4bd634..6e5e6a3 100644 --- a/.gitea/workflows/build.yml +++ b/.gitea/workflows/build.yml @@ -31,23 +31,24 @@ jobs: while IFS= read -r -d '' f; do ARGS+=(-i "$f") done < <(find "$SRC" -name "*.sqf" -print0) + # pipefail would kill this step on SQFVM's non-zero exit before we can check the log /tmp/sqfvm/sqfvm_linux_x64_gcc/sqfvm \ -a --parse-only --no-execute-print \ --input-config "$SRC/description.ext" \ - "${ARGS[@]}" 2>&1 | tee lint_output.log + "${ARGS[@]}" 2>&1 | tee lint_output.log || true # Filter known false positive: SQFVM config parser chokes on & in author string if grep -v "description.ext" lint_output.log | grep -q "^\[ERR\]"; then echo "::error::SQFVM found syntax errors" exit 1 fi - # - name: Upload lint log - # if: always() - # uses: actions/upload-artifact@v3 - # with: - # name: sqfvm-lint.log - # path: lint_output.log - # if-no-files-found: warn + - name: Upload lint log + if: always() + uses: actions/upload-artifact@v3 + with: + name: sqfvm-lint.log + path: lint_output.log + if-no-files-found: warn pack: needs: [lint]