Michał Krassowski

Results 1117 comments of Michał Krassowski

As of today hover is tested in `Hover.robot`: https://github.com/jupyter-lsp/jupyterlab-lsp/blob/7ae3c056ed5092b8081719138ead8d12902434e5/atest/05_Features/Hover.robot#L13-L45 This issue proposed to add a basic test of hover per-server. Given the simplicity of this feature I am not convinced...

We do not want to filter messages - we want to filter things shown in the completer; this is to allow instant completions from cache (this is discussed upstream in...

It does not preclude filtering messages - just not useful here because we want to have all completion items cached on the frontend.

@jasonm thank you for reporting. This is interesting problem. @bollwyvl Nah, not this javascript-typescript server, that one: ``` [D 2021-12-15 15:09:56.735 ServerApp] Checking for /home/jovyan/node_modules/typescript-language-server/lib/cli.js [D 2021-12-15 15:09:56.735 ServerApp] Checking...

No, it looks that your installation is fine; you could try to install the other typescript server (the old `javascript-typescript-langserver` rather than new `typescript-language-server`) to narrow down the issue though....

Could it be that `.virtual_documents` get created someplace else (other than `/home/jovyan/.virtual_documents`)?

It looks like we are currently testing against `typescript-language-server` v0.6.4 from October, but things are moving fast and they already released v0.8.1 and both 0.7 and 0.8 had breaking changes....

We can solve this relatively easily. This is because we already transform cell magics to equivalent Python commands; for example ``` %%script bash echo "2" ls ``` becomes: ```python get_ipython().run_cell_magic("script",...

I saw the pylsp comment, I don't quite agree that it should be implemented by the IDE. It seems pretty language-specific to me.

Also if it's due to bandwidth it has to go into the language server - we can only filter on frontend.