Repository navigation
Run the tests serially in CI - #201
Merged
Merged
Conversation
swift test runs the Swift Testing suites in parallel, and run that way the suite has crashed inside AppKit and InputMethodKit on an unchanged main, one run in eight (measured in #175); serially it held ten runs in ten there. The serial run takes about eight seconds, so nothing is saved by the parallel one. CONTRIBUTING names the same command, so a local run and CI agree. actions/checkout moves to v5: v4 targets Node.js 20, which the runners have deprecated and were already forcing onto Node.js 24. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
swift testruns the Swift Testing suites in parallel. Run that way, the suite has crashed inside AppKit/InputMethodKit on an unchangedmain: one run in eight, as measured in Expose two-way editor focus #175. Serially it held ten runs in ten there.actions/checkout@v4targets Node.js 20. The runners have deprecated it and were already forcing it onto Node.js 24 (annotation on the lastmainrun).What
swift test --no-parallel. The serial run takes about 8 s locally (600 tests in 78 suites), so the parallel one saves nothing.actions/checkout@v5(runs on Node 24).Checks
swift test --no-parallelon this branch: 600 tests in 78 suites, all pass.🤖 Generated with Claude Code