Skip to content

Commit a16a5e0

Browse files
ZviBaratzclaude
andcommitted
fix: remove local keyword from top-level review loop
`local` can only be used inside functions. The throttle block in the review phase runs at top level, not inside a function. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 5c3d46a commit a16a5e0

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

scripts/field-test-runner.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,8 @@ if [[ "$OPT_REVIEW" == true || "$OPT_REVIEW_CHANGED" == true ]]; then
404404
if [[ ${#REVIEW_PIDS[@]} -ge $OPT_PARALLEL ]]; then
405405
wait -n 2>/dev/null || true
406406
# Collect finished processes
407-
local new_pids=() new_names=()
407+
new_pids=()
408+
new_names=()
408409
for i in "${!REVIEW_PIDS[@]}"; do
409410
if kill -0 "${REVIEW_PIDS[$i]}" 2>/dev/null; then
410411
new_pids+=("${REVIEW_PIDS[$i]}")

0 commit comments

Comments
 (0)