Add code to handle incremental text updates about tower-lsp HOT 11 CLOSED

Finally got around to retrying this - haven't got incremental code changes to work yet (waiting to see what comes of brendanzab/codespan#249), but Go To Definition works perfectly, thanks!

from tower-lsp.

Thanks for the kind words and feedback! Have you thought of using codespan and codespan-lsp, by any chance? The latter crate in particular should handle most of the conversions you describe, and it's what I personally use in my own LSP projects, including incremental text updates. With that said, I'm open to potentially "baking in" this sort of functionality into the project as an optional addition.

As for textDocument/typeDefinition, I suppose that API addition will have to come in the next few releases. 😄

from tower-lsp.

Both textDocument/definition and textDocument/typeDefinition are on the list in #10. Hopefully I can knock them out over the next several days. If you would like to submit a pull request yourself, of course, that would also be most welcome.

from tower-lsp.

Thanks for the links to codespan - didn't know about that. Since I wrote this I also discovered that rust-analyzer has its LSP stuff as a generic crate. Here's an example. However they don't provide a nice trait like you do, so I'd definitely rather use this if it were complete.

from tower-lsp.

@Timmmm I've just merged in pull request #76, which implements goto declaration, goto definition, and goto type definition support. I hope to have a new release out on Crates.io fairly soon! Feel free to depend directly on the master branch in the meantime, if your project allows it and you're eager to begin iterating. ☺️

from tower-lsp.

Ah cool, thanks! I'll give it a go.

from tower-lsp.

@Timmmm Please alert me to any glaring issues come up before the next release! I'll do my best to get in some quick fixes before publishing to Crates.io, if deemed necessary.

from tower-lsp.

@Timmmm Version 0.6.0 has just been published to Crates.io with support for goto declaration, goto definition, and goto type definition.

from tower-lsp.

We are currently depending on an old version of lsp-types (0.68 vs. 0.70, at the time of writing) because codespan-lsp hasn't been updated to support the latest version of lsp-types (see brendanzab/codespan#163). Once this is updated, I think we can update our lsp-types dependency to the latest version and close this issue as invalid.

from tower-lsp.

Closed by #105, which gives us access to the latest lsp-types and therefore compatibility with the latest codespan-lsp.

from tower-lsp.

@Timmmm That's fantastic news! Glad it works for you. 🤗 I'm still waiting to see what the future of codespan will actually be, given that @brendanzab has been refactoring the repo quite a bit. I'm sure some solution for your needs will still exist though.

from tower-lsp.