Skip to content

Flush stdout/stderr between wasm-opt tests#8680

Merged
sbc100 merged 1 commit intomainfrom
flush
May 7, 2026
Merged

Flush stdout/stderr between wasm-opt tests#8680
sbc100 merged 1 commit intomainfrom
flush

Conversation

@sbc100
Copy link
Copy Markdown
Member

@sbc100 sbc100 commented May 7, 2026

Without this the github CI often shows the errors in confusing locations interspersed with the test names.

I confirmed in #8679 that this addresses the issue. Maybe there is a better place to do this but this is good improvement for now.

Without this the github CI often shows the errors in confusing locations
interspersed with the test names.

I confirmed in #8679 that this addresses the issue.  Maybe there is a
better place to do this but this is good improvement for now.
@sbc100 sbc100 requested a review from a team as a code owner May 7, 2026 00:11
@sbc100 sbc100 requested review from stevenfontanella and tlively and removed request for a team May 7, 2026 00:11
@sbc100 sbc100 enabled auto-merge (squash) May 7, 2026 00:20
@sbc100 sbc100 merged commit 6b06b59 into main May 7, 2026
16 checks passed
@sbc100 sbc100 deleted the flush branch May 7, 2026 01:03
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