Well, the latest commits are "geany-lsp" powered :).
It works quite nice, so far even without an active project :D.It's actually surprising (but I can reproduce it too). By default
use_without_project=false
but the server starts even with this set.Thanks for the "bug report" :-)
Oh, I wasn't clear. I had use_without_project=true
set, so in my case it was not so surprising. I used it this way on purpose.
The only drawback is that the project root (after having it detected using project_root_marker_patterns
), is sent to the server only on initialization. That is, if you switch to another file of the same language in another project root, the server doesn't know the new root directory.
Restarting the server helps and might be ok for that use case.
Anyway, I've just tested the PR briefly and it seems to do what I'd expect it to do. So apart from the cosmetic
TRUE
comment above (which I can see you have just fixed), the PR looks good to me.
@vvillenave @advice2020 do you have any chance to test this PR (i.e. compile Geany from source)?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.