From 6141a82d0527be5d3c11e3f2a557c01dc7df3c4e Mon Sep 17 00:00:00 2001 From: Christophe Dervieux Date: Thu, 9 Apr 2026 17:34:38 +0200 Subject: [PATCH] Fix CI step swallowing quarto list tools errors MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The 'Verify quarto list tools does not show chromium' step runs under bash -e (errexit). If quarto list tools exits non-zero (e.g. transient network failure in latestRelease()), the script dies at the command substitution before echo prints any output — leaving zero diagnostics. Use set +e around the command and check the exit code separately, matching the pattern already used by the install and update steps in the same workflow. --- .github/workflows/test-install.yml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/.github/workflows/test-install.yml b/.github/workflows/test-install.yml index 81ca3f743b..6b0d3ed5e4 100644 --- a/.github/workflows/test-install.yml +++ b/.github/workflows/test-install.yml @@ -123,8 +123,15 @@ jobs: - name: Verify quarto list tools does not show chromium shell: bash run: | + set +e output=$(quarto list tools 2>&1) + exit_code=$? + set -e echo "$output" + if [ "$exit_code" -ne 0 ]; then + echo "::error::'quarto list tools' exited with code $exit_code (see output above)" + exit 1 + fi if echo "$output" | grep -iq "chromium"; then echo "::error::Deprecated chromium should not appear in 'quarto list tools' when not installed" exit 1