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

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

Open
kunzaatko opened this issue Dec 11, 2021 · 7 comments

Comments

@kunzaatko
Copy link

kunzaatko commented Dec 11, 2021

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
Copy link
Author

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

@kunzaatko
Copy link
Author

@tjdevries
Copy link
Member

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?

@kunzaatko
Copy link
Author

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
Copy link
Author

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.

@j-hui
Copy link

j-hui commented Feb 9, 2022

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

@kunzaatko
Copy link
Author

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?

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

3 participants