Skip to content
This repository has been archived by the owner on Jul 6, 2021. It is now read-only.

Discussion: Might be able to re-use more built-in logic now #18

Open
tjdevries opened this issue May 11, 2020 · 4 comments
Open

Discussion: Might be able to re-use more built-in logic now #18

tjdevries opened this issue May 11, 2020 · 4 comments

Comments

@tjdevries
Copy link
Member

Hey, just drive-by issue of saying you might be able to re-use more of the built-in stuff for diagnostics now as we have exposed more stuff in master.

Also wondering if there was more stuff we could expose that would be helpful to you in this plugin.

Thanks.

@tjdevries tjdevries changed the title Discussin: Might be able to re-use more built-in logic now Discussion: Might be able to re-use more built-in logic now May 11, 2020
@haorenW1025
Copy link
Collaborator

Yeah actually I've been planing to of move some stuff out, but preserving backward compatibility while doing that is kind of hard...I'll start by moving some redundant stuff out and slowly integrate to the built-in stuff. Thanks for the reminder still:)

@clason
Copy link

clason commented May 12, 2020

I don't think you need to worry about backwards compatibility yet -- both neovim LSP and this plugin are in early access, so "here there be dragons" and all that... :)

(Maybe a fat disclaimer on top of the README to that effect might be helpful to manage expectations, though -- people tend to get excited when they see shiny new things!)

@haorenW1025
Copy link
Collaborator

@clason Make sense. Maybe I'll open a new branch and keep up with the latest built-in functions. Eventually it'll be merged into the master branch.

@aktau
Copy link

aktau commented Jul 7, 2020

Which functions are we talking about here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants