lsp_extensions.nvim icon indicating copy to clipboard operation
lsp_extensions.nvim copied to clipboard

[deprecated] Use of deprecated api in `lsp_extensions/workspace/diagnostic.lua`

Open kunzaatko opened this issue 4 years ago • 7 comments

vim.lsp.diagnostic.get is now being deprecated in favour of vim.diagnostic.get. There are more deprecations to be seen in :h deprecated.

kunzaatko avatar Dec 11 '21 11:12 kunzaatko

You can assign me to this issue however, I will have time only in about 2 weeks.

kunzaatko avatar Dec 11 '21 11:12 kunzaatko

akinsho/bufferline.nvim#272

kunzaatko avatar Dec 11 '21 11:12 kunzaatko

I think perhaps it's better to not even have this anymore. You can just use vim.diagnostic.get since that already does workspaces.

I think I should just remove it completely, what do you think?

tjdevries avatar Dec 16 '21 01:12 tjdevries

Yeah, I think this would actually be a good solution. Maybe there should be a notice in the README or just a deprecation warning for a while, for the people that are using this still?

kunzaatko avatar Dec 16 '21 14:12 kunzaatko

In the case of a deprecation warning, which is the best solution in my opinion, the section of the README, should be removed in the meantime.

kunzaatko avatar Dec 16 '21 14:12 kunzaatko

@kunzaatko did you get a chance to work on this?

It actually took me weeks to get around to figuring out where this deprecation warning was coming from; the actual vim.lsp.diagnostic.get_count is deprecated. See :h deprecated warning doesn't exactly tell me where it's coming from so I had to do a binary search through my plugins...

j-hui avatar Feb 09 '22 15:02 j-hui

Hi! Sorry for the delay. I did not work on this but as TJ said, there is no reason in using this anymore, if I am not mistaken. All of the features that were provided by the plugin are now in neovim itself. Am I correct?

kunzaatko avatar Mar 09 '22 09:03 kunzaatko