-
Notifications
You must be signed in to change notification settings - Fork 196
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
Exception in ElixirLS.LanguageServer.SourceFile #184
Comments
That's strange as elixir_sense master@f89444dd713520acb5d97a98b6d24386c04b9ae8 returns 3-element tuple and the updated code in |
Yes that is possible, I will double check in a bit.
…On Sun, Apr 5, 2020, 9:53 AM Łukasz Samson ***@***.***> wrote:
That's strange as elixir_sense
***@***.*** returns 3-element tuple
and the updated code in module_line/1 in #180
<#180> matches on that
3-element tuple.
Could it be that older version of ElixirSense.Core.Normalized.Code or the
whole elixir-LS is being loaded instead? I had a similar issue related to
elixir-lsp/vscode-elixir-ls#63
<elixir-lsp/vscode-elixir-ls#63> (I haven't
tested the fix yet, but 0.3.2 tag with
045875a applied works fine on my machine)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#184 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAACN5KSDEEKGS3NI6VTGQDRLDOSZANCNFSM4MBGUETA>
.
|
Yeah you were correct. I need to remember that when you're working on ElixirLS locally all the files that you are editing are replacing the modules in the version of ElixirLS that you're running as well. |
I'm seeing an exception in
ElixirLS.LanguageServer.SourceFile
:The exception is coming from the
get_docs
line:Most likely related to the elixir_sense update in #180
cc: @lukaszsamson
The text was updated successfully, but these errors were encountered: