Skip to content

feat: stream shorebird_tests output for CI debugging#106

Merged
eseidel merged 1 commit into
shorebird/devfrom
es/log_shorebird_tests
Feb 3, 2026
Merged

feat: stream shorebird_tests output for CI debugging#106
eseidel merged 1 commit into
shorebird/devfrom
es/log_shorebird_tests

Conversation

@eseidel

@eseidel eseidel commented Feb 3, 2026

Copy link
Copy Markdown

Summary

  • Switches _runFlutterCommand from Process.run (buffered) to Process.start (streaming) so stdout/stderr are printed in real time
  • Helps diagnose flaky timeout failures in android_test.dart on CI — if a process hangs, the logs will show exactly where it stopped

Test plan

  • Verify shorebird_tests CI job now shows streaming output for each flutter command

@eseidel eseidel requested a review from bdero February 3, 2026 00:46
@eseidel eseidel enabled auto-merge (squash) February 3, 2026 00:46
@eseidel

eseidel commented Feb 3, 2026

Copy link
Copy Markdown
Author

Example failure: https://github.com/shorebirdtech/flutter/actions/runs/21611184849/job/62280081866

Before I try fixes I want to see full logs, which this should give us.

@eseidel

eseidel commented Feb 3, 2026

Copy link
Copy Markdown
Author

The log format I (claude) picked is probably too verbose though, I may remove the prefix.

@eseidel eseidel merged commit 8f6ff05 into shorebird/dev Feb 3, 2026
3 of 5 checks passed
@eseidel eseidel deleted the es/log_shorebird_tests branch February 3, 2026 00:56
mrnpro pushed a commit to Flutter-Code-Push/flutter that referenced this pull request Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants