-
Notifications
You must be signed in to change notification settings - Fork 215
Allow configuring stdlib directories #1119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I ran into this issue. @lukaszsamson thanks for the PRs for elixir_sense. Is there a current workaround or feature branch with the changes to allow elixir_ls to read the source directory from an env var? |
No, it's not implemented on ElixirLS side. It shouldn't be difficult. Are you willing to submit a PR? |
thanks. Let me know if this works #1164 I haven't been able to test it, but i'll try later today |
Nice. Now we need a PR adding the new setting to the vscode extension in https://github.com/elixir-lsp/vscode-elixir-ls/blob/5b1c42df7c4e105bf646ddc17f95e93672c51248/package.json#L66 and it should be easy to test |
thanks, here it goes! elixir-lsp/vscode-elixir-ls#452 |
I also tested this PR with neovim and it works as expected :) |
Nice @cbecker. Merged |
This PR added necessary support to elixir_sense elixir-lsp/elixir_sense#277 (elixir only). We need to expose it to elixir-ls settings and make sure OTP works as well.
The text was updated successfully, but these errors were encountered: