mirror of https://github.com/garrytan/gstack.git
fix: add SC2059 to shellcheck disable in evals PR comment step
The SC2086 disable only covered the first command — the `for f in $RESULTS` loop and printf-style string building triggered SC2086 and SC2059 warnings. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
7b2225cac5
commit
e764dbb51f
|
|
@ -149,7 +149,7 @@ jobs:
|
||||||
env:
|
env:
|
||||||
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
|
||||||
run: |
|
run: |
|
||||||
# shellcheck disable=SC2086
|
# shellcheck disable=SC2086,SC2059
|
||||||
RESULTS=$(find /tmp/eval-results -name '*.json' 2>/dev/null | sort)
|
RESULTS=$(find /tmp/eval-results -name '*.json' 2>/dev/null | sort)
|
||||||
if [ -z "$RESULTS" ]; then
|
if [ -z "$RESULTS" ]; then
|
||||||
echo "No eval results found"
|
echo "No eval results found"
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue