ci: avoid flaky SwiftPM test runner hangs#6
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughGitHub Actions workflows for macOS CI and release jobs are updated to build Swift test targets instead of executing them. The CI workflow replaces ChangesTest Execution Strategy
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
Stabilize the macOS CI and release validation workflows after repeated GitHub runner hangs in the SwiftPM
swift testexecution path.Changes
swift teststep withswift build --build-tests, keeping SwiftPM package/test-target compilation coverage without launching the flakyswiftpm-testingrunner.swift test --parallellightweight smoke step from CI. The same suite is already covered by the Xcode test step, and the parallel SwiftPM runner was the timeout source.Validation
ruby -e 'require "yaml"; YAML.load_file(".github/workflows/ci.yml"); YAML.load_file(".github/workflows/release.yml")'git diff --checkswift build --package-path PitcherPlantApp --build-testsxcodebuild -project PitcherPlantApp/PitcherPlantApp.xcodeproj -scheme PitcherPlantApp -destination 'platform=macOS' test— 119 tests passed,TEST SUCCEEDEDNotes
The failing master runs timed out inside
swiftpm-testingafter many tests had already reported success. Xcode tests completed successfully in the same workflows, so this keeps the real test gate on the stable Xcode test path instead of repeating the suite through SwiftPM.Summary by CodeRabbit