Skip to content

Commit f86a0bb

Browse files
committed
Fix CI step swallowing quarto list tools errors
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.
1 parent 2742236 commit f86a0bb

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

.github/workflows/test-install.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,15 @@ jobs:
123123
- name: Verify quarto list tools does not show chromium
124124
shell: bash
125125
run: |
126+
set +e
126127
output=$(quarto list tools 2>&1)
128+
exit_code=$?
129+
set -e
127130
echo "$output"
131+
if [ "$exit_code" -ne 0 ]; then
132+
echo "::error::'quarto list tools' exited with code $exit_code (see output above)"
133+
exit 1
134+
fi
128135
if echo "$output" | grep -iq "chromium"; then
129136
echo "::error::Deprecated chromium should not appear in 'quarto list tools' when not installed"
130137
exit 1

0 commit comments

Comments
 (0)