elixir-lsp / vscode-elixir-ls

Elixir language support and debugger for VS Code, powered by ElixirLS.
https://marketplace.visualstudio.com/items?itemName=JakeBecker.elixir-ls
MIT License
545 stars 105 forks source link

dialyzerEnabled flag config is ignored after ElixirLS is restarted #366

Closed sasa1977 closed 1 year ago

sasa1977 commented 1 year ago

Is this the right repo?

I think so, since the issue is about a vscode config being ignored. But I might be wrong.

Environment

Troubleshooting

Current behaviour

I have "elixirLS.dialyzerEnabled": false in vscode settings. When I start vscode, the dialyzer step is correctly skipped. However, if I restart the language server (using the "Restart language server" command), the flag is ignored, and dialyzer is running on every compilation. The dialyzer warnings are now present in the problems tab.

Expected behaviour

After restart, the language server should honor the elixirLS.dialyzerEnabled config and skip the dialyzer step.

michallepicki commented 1 year ago

I am affected by this issue whenever the language server encounters an error and crashes (and is automatically restarted).

lukaszsamson commented 1 year ago

It looks like another manifestation of https://github.com/elixir-lsp/elixir-ls/issues/921

lukaszsamson commented 1 year ago

The original issue is fixed an now on restart the server will call workspace/configuration to get configuration. There is still a bug in VSCode - after restart the server no longer receives notifications about configuration changes https://github.com/elixir-lsp/vscode-elixir-ls/issues/368

lukaszsamson commented 1 year ago

BTW this is a recent breaking change in VSCode and it affects other language servers (e.g. https://github.com/haskell/vscode-haskell/issues/920)