vieiro commented on PR #3978: URL: https://github.com/apache/netbeans/pull/3978#issuecomment-1101595136
Hi @matthiasblaesing , thanks for the review. 1.- I think it's ok to expose the state of the LSP server. I think developers want to know if things are running or not (in the cpplite era one could not see if the lsp server was running or not). 2.- Regarding the changes, we can split them out and send them to master. I thought it would be a good idea to make a test in cnd first, with two LSP servers running at the time. Anyway making all different capabilities go though a single RequestProcessor solves issue https://github.com/apache/netbeans/issues/3812 . Best we can do, I think, is to test here in this branch (without merging) with two LSP servers running (say clangd and typescript), see if things run well, and then remove the commit from this PR and send it to master instead. What say? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
