From dce12e0bb5abcba872e714535b578abe83dbe9da Mon Sep 17 00:00:00 2001 From: cluade-reviewer-andresmgsl Date: Sun, 2 Aug 2026 19:24:27 +0000 Subject: [PATCH] fix(test): the second curl stub needed the SC2317 disable too MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit a968e13 was pushed with shellcheck red. I chained the gates and the push in one command, so a non-zero gate did not stop the push — the gate has to be a condition, not a line of output I read afterwards. Refs #188 --- test/forge-backends.test.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/test/forge-backends.test.sh b/test/forge-backends.test.sh index d4c6ce4..72fd95a 100644 --- a/test/forge-backends.test.sh +++ b/test/forge-backends.test.sh @@ -204,6 +204,7 @@ check "an empty collection is not an error" 0 "" \ # gh exits non-zero on an HTTP error; curl does not without -f, and -f # discards the body that explains why. So the status is read explicitly. fake_forge 1 '[{"number":1}]' +# shellcheck disable=SC2317 # invoked indirectly, by forge_api curl() { local hdr="" out="" while [ $# -gt 0 ]; do