Skip to content

fix: preserve corpus timeout output - #602

Merged
LunaStev merged 1 commit into
wavefnd:masterfrom
nightcityblade:fix/issue-593
Sep 12, 2026
Merged

fix: preserve corpus timeout output#602
LunaStev merged 1 commit into
wavefnd:masterfrom
nightcityblade:fix/issue-593

Conversation

@nightcityblade

Copy link
Copy Markdown
Contributor

Summary

Preserve compiler stdout and stderr when a corpus check times out.

Motivation

Fixes #593.

Target and compatibility impact

None; diagnostics now match the existing runnable-example timeout behavior.

Validation

  • python3 -m unittest tools.test_check_wave_corpus (13 passed)
  • python3 -m ruff check tools/check_wave_corpus.py tools/test_check_wave_corpus.py
  • git diff --check

Checklist

  • Commits include a DCO Signed-off-by line.
  • Tests cover new behavior or the PR explains why no test is needed.
  • User-facing changes include documentation or diagnostics updates.
  • The change preserves the license boundary between the compiler and std/.

Signed-off-by: nightcityblade <nightcityblade@gmail.com>
@LunaStev

Copy link
Copy Markdown
Member

The implementation is LGTM; just a minor issue with the tests.

But I'll merge it anyway.

@LunaStev
LunaStev merged commit bb8b9a4 into wavefnd:master Sep 12, 2026
23 of 26 checks passed
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.

Preserve timeout output in the corpus checker

2 participants