Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace deprecated checkhealth syntax with newer syntax #1117

Merged
merged 1 commit into from
Dec 30, 2023

Conversation

dundargoc
Copy link
Contributor

No description provided.

@davidhalter
Copy link
Owner

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

@dundargoc
Copy link
Contributor Author

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

@davidhalter
Copy link
Owner

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

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 davidhalter merged commit 9bd79ee into davidhalter:master Dec 30, 2023
2 of 3 checks passed
@davidhalter
Copy link
Owner

Thanks!

@dundargoc dundargoc deleted the checkhealth branch December 30, 2023 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants