-
Notifications
You must be signed in to change notification settings - Fork 129
WIP: Extended CI #150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: Extended CI #150
Conversation
This opts TSC in to running SwiftPM and SourceKit-LSP tests as well. Since there are some CI-specific issues to work through, we need a PR to iterate on them.
Continuation of #148 |
swiftlang/swift-package-manager#2993 |
Linux issue is that any tests that are running an inferior |
The macOS job is consistenly hanging in |
I wonder if the hang is related to what we were seeing in swiftlang/swift-package-manager#2842 |
Going to try |
@swift-ci please test |
4 similar comments
@swift-ci please test |
@swift-ci please test |
@swift-ci please test |
@swift-ci please test |
|
@swift-ci please test macOS |
Nice, looks like using |
@swift-ci please test linux |
1 similar comment
@swift-ci please test linux |
Seems as if the Linux build is broken somehow:
Not sure why we aren't seeing the same thing in SwiftPM's job, both should be identical now. |
@swift-ci please test Linux platform |
@swift-ci please test Linux |
I understand the linux issue now, the CI script was using the macOS preset also for Linux will led to system ICU being used. |
@swift-ci please test Linux |
Great this passes now, so we can enable the extended CI in general and close this PR. |
OK, this can be closed now. |
This opts TSC in to running SwiftPM and SourceKit-LSP tests as well. Since there are some CI-specific issues to work through, we need a PR to iterate on them.