Skip to content

Commit b5c095f

Browse files
sbryngelsonclaude
andcommitted
Remove redundant NUM_FAILED > 0 guard in test retry logic
The -s check already guarantees the file is non-empty, so NUM_FAILED > 0 is always true in that branch. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent aa26048 commit b5c095f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ jobs:
151151
# Retry only if a small number of tests failed (sporadic failures)
152152
if [ -s tests/failed_uuids.txt ]; then
153153
NUM_FAILED=$(wc -l < tests/failed_uuids.txt)
154-
if [ "$NUM_FAILED" -gt 0 ] && [ "$NUM_FAILED" -le 5 ]; then
154+
if [ "$NUM_FAILED" -le 5 ]; then
155155
FAILED=$(cat tests/failed_uuids.txt | tr '\n' ' ')
156156
echo ""
157157
echo "=== Retrying $NUM_FAILED failed test(s): $FAILED ==="

0 commit comments

Comments
 (0)