Siddharth Srinivasan
Siddharth Srinivasan
> Looks OK to me. Thanks! Thank you very much.
The LSP `ServerTest. testCodeFolding` and `ServerTest.testPullUp` seem to be failing in the merge tests run due to an NPE since `NbCodeClientCapabilities.getClientCapabilities()` is `null` for the unit tests. I was trying...
Thank you very much @lahodaj.
Hi @mbien. Please review this PR and/or assign other reviewers, as per your convenience. Thanks.
Thank you @lahodaj for reviewing the PR. > So, what if, just before the new code here, we would check if there's exactly one workspace, and if there is, we...
Thanks @lahodaj for the explanation of the concrete implementation context. The key differentiation in the patch is that a workspace with ***no*** folders is chosen to associate the single-file with....
Understood. It makes sense to keep this simple. I'll revise the patch and add to this PR. Thank you very much @lahodaj.
I've pushed the patch incorporating the suggestion above by @lahodaj. Please review. Thanks.
> don't forget to squash before merge Thanks @mbien. I've squashed the commits.
Requesting a merge of this PR. Thank you.