From e764dbb51f82299988bf9cd03d173bbd0bd81923 Mon Sep 17 00:00:00 2001
From: Garry Tan <garrytan@gmail.com>
Date: Mon, 23 Mar 2026 16:08:26 -0700
Subject: [PATCH] fix: add SC2059 to shellcheck disable in evals PR comment
 step
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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>
---
 .github/workflows/evals.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/evals.yml b/.github/workflows/evals.yml
index 8c749322..162ee8f0 100644
--- a/.github/workflows/evals.yml
+++ b/.github/workflows/evals.yml
@@ -149,7 +149,7 @@ jobs:
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
-          # shellcheck disable=SC2086
+          # shellcheck disable=SC2086,SC2059
           RESULTS=$(find /tmp/eval-results -name '*.json' 2>/dev/null | sort)
           if [ -z "$RESULTS" ]; then
             echo "No eval results found"