From 36cf64b15ff9cdf54657ebd6531ffd07614117f8 Mon Sep 17 00:00:00 2001 From: Will Lin Date: Fri, 3 Jul 2026 23:49:44 +0000 Subject: [PATCH] [bugfix]: fix bash syntax error in modal ci pr checkout MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The multi-line retry checkout command ended with a newline, so splicing it into '{checkout_command} &&' left a lone '&&' on the line after 'done' — a bash syntax error failing every PR-triggered shard before checkout. End the f-string at 'done' so the composed script reads 'done &&'; verified all four compositions (PR/direct x kernel/no-kernel) with bash -n. Co-Authored-By: Claude Fable 5 --- fastvideo/tests/modal/pr_test.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fastvideo/tests/modal/pr_test.py b/fastvideo/tests/modal/pr_test.py index 8416a1cfa..43a91ae96 100644 --- a/fastvideo/tests/modal/pr_test.py +++ b/fastvideo/tests/modal/pr_test.py @@ -121,8 +121,7 @@ def run_test_command(test_command: str, exit "$status" fi sleep $((attempt * 5)) - done - """ + done""" print(f"Using PR ref for checkout: {checkout_command}") else: checkout_command = f"git checkout {git_commit}"