-
Notifications
You must be signed in to change notification settings - Fork 325
Install SourceKit-LSP configuration schema into toolchain at share/sourcekit-lsp/config.schema.json
#2147
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
base: main
Are you sure you want to change the base?
Conversation
…ourcekit-lsp/config.schema.json` This allows IDEs like VS Code to pick up the schema and provide code assistance when editing `.sourcekit-lsp/config.json`.
|
@swift-ci Please test |
|
@swift-ci Please test macOS |
compnerd
left a comment
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.
Can you please add an install rule in the CMake build for this as well?
|
That’s the reason why the PR hasn’t been merged for half a year 😉 |
|
That comment was in response to me putting up a build.ps1 change instead of the cmake like I should have :) I'll put that up here |
|
ahoppen
left a comment
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 for taking care of the Windows install. CMake change looks good to me.
This allows IDEs like VS Code to pick up the schema and provide code assistance when editing
.sourcekit-lsp/config.json.