-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Sync latest TSC changes from main #2965
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
Conversation
@swift-ci please smoke test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I reached out to Mishal, since it seems like an infra issue. |
Ah, yes, that is fallout of the branch renaming. |
Looks like that's a new file I didn't add as part of the sync. |
@swift-ci please smoke test |
Looks like SourceKit-LSP might need an update following the TSC change. |
@neonichu swiftlang/sourcekit-lsp#325 should fix that (and is what uncovered this in the first place) |
Please test with following PRs: @swift-ci please smoke test |
Please test with following PRs: @swift-ci please smoke test |
Seems as if that's just some flakiness and not an issue with the PR. |
Please test with following PRs: @swift-ci please smoke test |
@neonichu - can we get this merged? the windows enumerated value is in the version of TSC that LSP is now building against. |
Please test with following PRs: @swift-ci please smoke test |
No description provided.