Skip to content

Commit b1ab17b

Browse files
committed
ci(mobile-e2e): keep Android while-loop on one line for folded YAML
The folded scalar (>-) only joins same-indent lines with spaces; more-indented lines preserve newlines, which broke the shell's while/do/done pairing and produced 'end of file unexpected expecting done'.
1 parent 9f29097 commit b1ab17b

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

.github/workflows/mobile-e2e.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -540,16 +540,17 @@ jobs:
540540
# rest) AND tag filtering, so pre-filter the file list in shell:
541541
# for each candidate flow, grep its YAML for a top-level
542542
# `- <excluded-tag>` line and drop it from the list.
543+
# All lines at the same indent so YAML's folded scalar (>-) joins
544+
# them with single spaces. Any extra indentation would preserve
545+
# newlines and break the shell pipe. The while-do-done stays on
546+
# one logical line for the same reason.
543547
script: >-
544548
adb install -r /tmp/cached-app-release.apk &&
545549
cd integration/mobile &&
546550
excluded="$EXCLUDE_TAGS,flakyAndroid" &&
547551
pattern="$(echo "$excluded" | sed 's/,/|/g')" &&
548-
find flows -type f -name '*.yaml' ! -path '*/common/*'
549-
${FLOWS_FILTER:+-path "*$FLOWS_FILTER*"}
550-
| while read f; do
551-
grep -qE "^[[:space:]]*-[[:space:]]*(${pattern})[[:space:]]*$" "$f" || printf '%s\n' "$f";
552-
done
552+
find flows -type f -name '*.yaml' ! -path '*/common/*' ${FLOWS_FILTER:+-path "*$FLOWS_FILTER*"}
553+
| while read f; do grep -qE "^[[:space:]]*-[[:space:]]*(${pattern})[[:space:]]*$" "$f" || printf '%s\n' "$f"; done
553554
| xargs -n 1 maestro test --env CLERK_TEST_EMAIL="$CLERK_TEST_EMAIL" --env CLERK_TEST_PASSWORD="$CLERK_TEST_PASSWORD" --flatten-debug-output
554555
555556
- name: Upload Maestro artifacts on failure or cancel

0 commit comments

Comments
 (0)