Christian Schneider
Christian Schneider
Regarding cancellation: https://user-images.githubusercontent.com/8513883/131699130-13dcbc0a-dbac-4aec-b50d-c485a85d70cd.mp4 I achieved this by changing https://github.com/langium/langium/blob/2f18b81256d1ca475eea9b8da955cb156517522a/packages/langium/src/lsp/language-server.ts#L129-L139 to ``` export function addDocumentHighlightsHandler(connection: Connection, services: LangiumServices): void { const documentHighlighter = services.lsp.DocumentHighlighter; connection.onDocumentHighlight((params: DocumentHighlightParams, canceled: CancellationToken): Promise => {...
My findings regarding progressReporting and partialResult: The client has to provide corresponding tokens in the particular request messages, like shown in https://microsoft.github.io/language-server-protocol/specifications/specification-current/#partialResults I guess there is no rule on how...
Hi Tom, thanks for this idea. Instead of sticking to the classic append-based API: Did you give the `expandToNode`-based approach a try? That way you can write ``` function generateMethod(node:...
While reviewing and testing your changes of #1286 I could produce a corrupt state of `langiumDocuments by renaming a document's parent folder and then opening the document, see screenshots. Thus...
> workspace/didChangeWorkspaceFolders is for the case when you actually change the workspace by adding a completely new workspace folder or remove one from the workspace. Note that workspace folders are...
> The remaining question is how to fix the issue reported here. My current understanding is that we should use the file system watcher for that by adding a suitable...
> That's true, we should either enable and support debug for all situations, or disable the button altogether. +1
Just a general comment from outside as I drifted over here from https://github.com/kieler/KLighD/pull/149 Does it really make sense to re-arrange and relocate labels by moving over the text and deleting...
As indicated by the original title, I've prepared another change bumping to `yeoman-generator@^7.0.0` (which took also quite notable time because of some nasty details). However, some transitive dependency requires `node...
Since the ci build is now using Node18 (#193) I updated this branch and added my additional commit bumping the dependency of `yeoman-generator` to version `7.0.0`.