Skip to content

updated health to use new nvim health api#33

Open
mrdandelion6 wants to merge 1 commit intoGCBallesteros:mainfrom
mrdandelion6:main
Open

updated health to use new nvim health api#33
mrdandelion6 wants to merge 1 commit intoGCBallesteros:mainfrom
mrdandelion6:main

Conversation

@mrdandelion6
Copy link

changed all vim.health.report_* to vim.health.* to work with nvim version 11

@vandalt vandalt mentioned this pull request Apr 1, 2025
@bkp5190
Copy link

bkp5190 commented May 12, 2025

Can this please be merged? 😀

@DmitriyBalmashnov
Copy link

In the telescope repo I saw that the merged solution worked as follows:

local health = vim.health or require "health"
local start = health.start or health.report_start
local ok = health.ok or health.report_ok
local warn = health.warn or health.report_warn
local error = health.error or health.report_error

And then locally using start() etc.
I feel like maybe this can be done here too? So that the plugin does still work for people with a lower nvim version.

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.

3 participants