Skip to content

Commit 80a114d

Browse files
Trotttargos
authored andcommitted
tools: be intentional about masking possible error in start-ci.sh
Revise start-ci.sh to conform with shellcheck 0.8.0 default checks. Refs: https://www.shellcheck.net/wiki/SC2312 PR-URL: #41284 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent e819685 commit 80a114d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/actions/start-ci.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ for pr in "$@"; do
1717
# Do we need to reset?
1818
gh pr edit "$pr" --add-label "$REQUEST_CI_FAILED_LABEL"
1919

20-
jq -n --arg content "<details><summary>Couldn't start CI</summary><pre>$(cat output)</pre></details>" > output.json
20+
jq -n --arg content "<details><summary>Couldn't start CI</summary><pre>$(cat output || true)</pre></details>" > output.json
2121

2222
gh pr comment "$pr" --body-file output.json
2323

0 commit comments

Comments
 (0)