From 20a9d4eaea6f0e5bdeabd82756a9e7ebee34a14e Mon Sep 17 00:00:00 2001 From: VU3ESV Date: Sat, 6 Jun 2026 07:36:25 +0200 Subject: [PATCH] ci: fix gate step exit code (was failing the release) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The gate's final '[ "$skip" = true ] && echo …' exited non-zero when skip=false, failing the step; via needs: gate that skipped the release for real changes. Use an if-block + harden the PR files API call with || true. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/release.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index d9187a2..a66c141 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -67,7 +67,7 @@ jobs: esac files="$(gh api --paginate "repos/${{ github.repository }}/compare/${{ github.event.before }}...${{ github.event.after }}" -q '.files[].filename' || true)" elif [ "$ev" = "pull_request" ]; then - files="$(gh api --paginate "repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files" -q '.[].filename')" + files="$(gh api --paginate "repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}/files" -q '.[].filename' || true)" else echo "skip=false" >> "$GITHUB_OUTPUT"; exit 0 fi @@ -83,7 +83,7 @@ jobs: esac done <<< "$files" echo "skip=$skip" >> "$GITHUB_OUTPUT" - [ "$skip" = true ] && echo "::notice::Docs-only change — skipping release." + if [ "$skip" = true ]; then echo "::notice::Docs-only change — skipping release."; fi release: needs: gate