Oscar
Oscar
@heywhy Thank you for the effort! I'll be trying it, mainly in Neovim v0.11.4. Just to confirm, is the fix also solving the "typing fast causes reordering problem"? For now,...
@mhanberg Just in case it helps, I've been unable to reproduce the problem in VSCode. With the same `expert` binary being used as LSP in both cases and typing at...
Hello! I've also been testing a bit, and it looks indeed much better in Neovim. Thank you for the effort! Still, I'm not sure if it's only me, but I...
Yes, I did. It might be just me if nobody else can reproduce it.
Sure! So I reproduce it by following these steps: 1. Create a new Phoenix project with `mix phx.new --no-ecto expert_testing` 2. Create a new module inside the project `lib/expert_testing/my_module.ex` 3....
Hi @mhanberg ! I have some time that I can dedicate to this. But I would like to confirm if: 1. This is also experienced by others and is not...
Completely agree. I just wanted to start a bit of the discussion, but I was indeed thinking that this might be a good "configurable" behaviour. I can take a look...
Yeah, just pointing it out as it is the default client behaviour in those two clients. I've checked the spec and it looks like there is already a `context` param...
Hey @john-eevee ! Thank you for checking. I think that, indeed, returning the definition is good behaviour, and even more if the LSP (and [Expert itself](https://github.com/elixir-lang/expert/blob/555b6b669dd348fc20cd643fef2cd3a43f7133b4/apps/engine/lib/engine/code_intelligence/references.ex#L15) itself) support an `includeDeclaration`...