diagnostic-nvim icon indicating copy to clipboard operation
diagnostic-nvim copied to clipboard

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

Open tjdevries opened this issue 5 years ago • 4 comments

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 avatar May 11 '20 16:05 tjdevries

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:)

haorenW1025 avatar May 11 '20 17:05 haorenW1025

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!)

clason avatar May 12 '20 07:05 clason

@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.

haorenW1025 avatar May 13 '20 01:05 haorenW1025

Which functions are we talking about here?

aktau avatar Jul 07 '20 18:07 aktau