Skip to content
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

lsp: Send non-null workspaceFolders in initialize #25337

Merged
merged 1 commit into from
Feb 21, 2025

Conversation

osiewicz
Copy link
Contributor

@osiewicz osiewicz commented Feb 21, 2025

This is a ~workaround for next-ls not handling null workspace folders in initialize request
Related to #25264
/cc @timfjord
Closes #ISSUE

Release Notes:

  • Changed how workspace folders are shared with language servers, fixing a startup issue with next-ls in the process.

This is a ~workaround for next-ls not handling null workspace folders in initialize request
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 21, 2025
@osiewicz osiewicz merged commit a97a2eb into main Feb 21, 2025
13 checks passed
@osiewicz osiewicz deleted the send-empty-workspace-folders branch February 21, 2025 16:05
@osiewicz
Copy link
Contributor Author

/cherry-pick v0.175.x

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Feb 21, 2025
This is a ~workaround for next-ls not handling null workspace folders in
initialize request
Related to #25264
/cc @timfjord
Closes #ISSUE

Release Notes:

- Changed how workspace folders are shared with language servers, fixing
a startup issue with `next-ls` in the process.
osiewicz added a commit that referenced this pull request Feb 21, 2025
#25344)

Cherry-picked lsp: Send non-null workspaceFolders in initialize (#25337)

This is a ~workaround for next-ls not handling null workspace folders in
initialize request
Related to #25264
/cc @timfjord
Closes #ISSUE

Release Notes:

- Changed how workspace folders are shared with language servers, fixing
a startup issue with `next-ls` in the process.

Co-authored-by: Piotr Osiewicz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant