-
Notifications
You must be signed in to change notification settings - Fork 27
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
Become LSP Provider #32
Comments
An LSP is somewhat different than an autocomplete provider. It actually has to implement the features of the protocol - so the end result might give both results from Also about making the server - I thought it might be possible to use an approach like none-ls and hook into the LSP features instead of writing a server. I realize this could take a lot and and your solution works perfectly (actually started using it today), but I was crious to hear if this sounds like something worth investing into. |
Thanks. Didn't they stop null-ls because it was getting hard to manage? Anyway I'll look into it and see if it's feasible. |
I second this approach as well. I think not only would it allow us to use the same conventional LSP mappings, but also allow faster & more concentrated navigation because |
Hi @xdavidel / @xquilt
Would that be enough to call it "lsp"? |
Well, according to the capabilities documentation I would argue Yes. Seems to me other capabilities can be achieve using some work and combine with clangd / ccls / etc. we will have a complete solution. |
cool. i'll start the implementation |
@xdavidel / @xquilt I have implemented Try it out and provide feedback |
That was quick. P.S: kudos for using TJs starting point for LSP |
@dhananjaylatkar Thanks for the work. I've tested it on a dir composed of multiple Android projects and it does work, but there are a couple of gotcha
I was wondering if go is your language of choice or if is it just because of the skeleton project by TJ. If you can consider Rust, I'd love to contribute to the project |
@xquilt Thanks for testing. Can you open issues on cscope_lsp repo? We can discuss there. Yeah, this is the first time I'm writing a "big" golang project. But also I've been looking for an opportunity to write golang so it works out for me. I don't see any upside of moving to Rust and I prefer the simplicity of golang so, I won't be doing rewrite in Rust anytime soon. |
Closing this issue. |
I'm really glad I was able to find this awesome plugin!
I was just doing some searching before writing my own solution to this problem.
My way of thinking led me to think about creating an LSP server for
cscope
DBs so it can be use alongsideclangd
or some other provider.Would you be interested in this kind of change?
Either way - this is a cool solution and I'm definitely using it from now on.
The text was updated successfully, but these errors were encountered: