Skip to content

Tidy up collab-related signature help data#14377

Merged
SomeoneToIgnore merged 1 commit intomainfrom
kb/better-remote-signatures
Jul 13, 2024
Merged

Tidy up collab-related signature help data#14377
SomeoneToIgnore merged 1 commit intomainfrom
kb/better-remote-signatures

Conversation

@SomeoneToIgnore
Copy link
Copy Markdown
Contributor

Follow-up of #12909

  • Fully preserve LSP data when sending it via collab, and only strip it on the client.
  • Avoid extra custom request handlers, and extend multi LSP server query protocol instead.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jul 13, 2024
* Fully preserve LSP data when sending it via collab, and only strip it on the client.
* Avoid extra custom request handlers, and extend multi LSP server query protocol instead.
@SomeoneToIgnore SomeoneToIgnore force-pushed the kb/better-remote-signatures branch from e696b45 to 5ab3482 Compare July 13, 2024 00:54
@SomeoneToIgnore
Copy link
Copy Markdown
Contributor Author

https://github.com/zed-industries/zed/actions/runs/9915970307/job/27397634196?pr=14377 fails due to breaking schema changes, but the PR alters unreleased part of it, hence it's safe to merge.

@SomeoneToIgnore SomeoneToIgnore merged commit 9ce989a into main Jul 13, 2024
@SomeoneToIgnore SomeoneToIgnore deleted the kb/better-remote-signatures branch July 13, 2024 01:06
SomeoneToIgnore added a commit that referenced this pull request Jul 15, 2024
Follow-up of #12909

* Fully preserve LSP data when sending it via collab, and only strip it
on the client.
* Avoid extra custom request handlers, and extend multi LSP server query
protocol instead.


Release Notes:

- N/A
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