From 5545674e6d4d9d0bbc58c33252bc7d0d00d3975b Mon Sep 17 00:00:00 2001 From: tytan652 Date: Sun, 7 Jul 2024 11:00:52 +0200 Subject: [PATCH] CI: Fix flatpak-builder-lint action - Fix for loops syntax - Manage jq empty output --- .github/actions/flatpak-builder-lint/action.yaml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/actions/flatpak-builder-lint/action.yaml b/.github/actions/flatpak-builder-lint/action.yaml index 2defa0124..86cd71793 100644 --- a/.github/actions/flatpak-builder-lint/action.yaml +++ b/.github/actions/flatpak-builder-lint/action.yaml @@ -51,15 +51,16 @@ runs: exit $exit_code fi - for ((i = 0 ; i < $(echo $ret | jq '.warnings | length') ; i++)); do + n_warnings=$(echo $ret | jq '.warnings | length') + for ((i = 0 ; i < n_warnings ; i++)); do warning=$(echo $ret | jq ".warnings[$i]") echo "::warning::$warning found in the Flatpak ${{ inputs.artifact }}" done n_errors=$(echo $ret | jq '.errors | length') - for ((i = 0 ; i < $n_errors ; i++)); do + for ((i = 0 ; i < n_errors ; i++)); do error=$(echo $ret | jq ".errors[$i]") echo "::error::$error found in the Flatpak ${{ inputs.artifact }}" done - [[ $n_errors == 0 ]] || exit 2 + [[ -z $n_errors || $n_errors == 0 ]] || exit 2