davidhalter / jedi-vim

Using the jedi autocompletion library for VIM.
MIT License
5.27k stars 370 forks source link

Replace deprecated checkhealth syntax with newer syntax #1117

Closed dundargoc closed 7 months ago

davidhalter commented 8 months ago

Can we do this without lua? My current VIM for example doesn't have lua support.

dundargoc commented 8 months ago

:checkhealth doesn't even work on vim since it's a neovim-exclusive command, and neovim always ships with lua.

davidhalter commented 7 months ago

Sorry, this is not code I have written. I'm trying my best to understand.

So do I understand this correctly: This whole function is only about nvim and will never be invoked by VIM? I don't understand the mechanism.

dundargoc commented 7 months ago

This whole function is only about nvim and will never be invoked by VIM?

Yeah. Try running :checkhealth jedi in vim. Nothing will happen. This is a neovim-only functionality so it should be fine to use lua functionality through vimscript as lua is shipped with neovim.

davidhalter commented 7 months ago

Thanks!