Skip to content

Commit

Permalink
fix(health): remove deprecated api warnings in health check (#2478)
Browse files Browse the repository at this point in the history
  • Loading branch information
escwxyz committed Apr 23, 2023
1 parent d95a812 commit f2645c1
Showing 1 changed file with 17 additions and 11 deletions.
28 changes: 17 additions & 11 deletions lua/telescope/health.lua
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
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
local info = health.info or health.report_info

local extension_module = require "telescope._extensions"
local extension_info = require("telescope").extensions
local is_win = vim.api.nvim_call_function("has", { "win32" }) == 1
Expand Down Expand Up @@ -60,33 +66,33 @@ local M = {}

M.check = function()
-- Required lua libs
health.report_start "Checking for required plugins"
start "Checking for required plugins"
for _, plugin in ipairs(required_plugins) do
if lualib_installed(plugin.lib) then
health.report_ok(plugin.lib .. " installed.")
ok(plugin.lib .. " installed.")
else
local lib_not_installed = plugin.lib .. " not found."
if plugin.optional then
health.report_warn(("%s %s"):format(lib_not_installed, plugin.info))
warn(("%s %s"):format(lib_not_installed, plugin.info))
else
health.report_error(lib_not_installed)
error(lib_not_installed)
end
end
end

-- external dependencies
-- TODO: only perform checks if user has enabled dependency in their config
health.report_start "Checking external dependencies"
start "Checking external dependencies"

for _, opt_dep in pairs(optional_dependencies) do
for _, package in ipairs(opt_dep.package) do
local installed, version = check_binary_installed(package)
if not installed then
local err_msg = ("%s: not found."):format(package.name)
if package.optional then
health.report_warn(("%s %s"):format(err_msg, ("Install %s for extended capabilities"):format(package.url)))
warn(("%s %s"):format(err_msg, ("Install %s for extended capabilities"):format(package.url)))
else
health.report_error(
error(
("%s %s"):format(
err_msg,
("`%s` finder will not function without %s installed."):format(opt_dep.finder_name, package.url)
Expand All @@ -95,13 +101,13 @@ M.check = function()
end
else
local eol = version:find "\n"
health.report_ok(("%s: found %s"):format(package.name, version:sub(0, eol - 1) or "(unknown version)"))
ok(("%s: found %s"):format(package.name, version:sub(0, eol - 1) or "(unknown version)"))
end
end
end

-- Extensions
health.report_start "===== Installed extensions ====="
start "===== Installed extensions ====="

local installed = {}
for extension_name, _ in pairs(extension_info) do
Expand All @@ -112,11 +118,11 @@ M.check = function()
for _, installed_ext in ipairs(installed) do
local extension_healthcheck = extension_module._health[installed_ext]

health.report_start(string.format("Telescope Extension: `%s`", installed_ext))
start(string.format("Telescope Extension: `%s`", installed_ext))
if extension_healthcheck then
extension_healthcheck()
else
health.report_info "No healthcheck provided"
info "No healthcheck provided"
end
end
end
Expand Down

0 comments on commit f2645c1

Please sign in to comment.