mirror of
https://github.com/neovim/neovim.git
synced 2024-12-23 20:55:18 -07:00
feat(vim.deprecate): only issue warning if neovim version is high enough
As specified by MAINTAIN.md, features should be soft deprecated at first (meaning no warnings) to give people a chance to adjust. The problem with this approach is that deprecating a feature becomes harder than usual as during the soft deprecation period you need to remember not to issue a warning, and during the hard deprecation period you need to remember to start issuing a warning. This behavior is only enforced if the `plugin` parameter is `nil` as plugins may not want this specific behavior.
This commit is contained in:
parent
343436f97d
commit
0a598c13b1
@ -1019,6 +1019,18 @@ end
|
|||||||
---
|
---
|
||||||
---@return string|nil # Deprecated message, or nil if no message was shown.
|
---@return string|nil # Deprecated message, or nil if no message was shown.
|
||||||
function vim.deprecate(name, alternative, version, plugin, backtrace)
|
function vim.deprecate(name, alternative, version, plugin, backtrace)
|
||||||
|
-- Only issue warning if feature is hard-deprecated as specified by MAINTAIN.md.
|
||||||
|
if plugin == nil then
|
||||||
|
local current_version = vim.version()
|
||||||
|
local deprecated_version = assert(vim.version.parse(version))
|
||||||
|
local soft_deprecated_version =
|
||||||
|
{ deprecated_version.major, deprecated_version.minor - 1, deprecated_version.patch }
|
||||||
|
local deprecate = vim.version.lt(current_version, soft_deprecated_version)
|
||||||
|
if deprecate then
|
||||||
|
return
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
local msg = ('%s is deprecated'):format(name)
|
local msg = ('%s is deprecated'):format(name)
|
||||||
plugin = plugin or 'Nvim'
|
plugin = plugin or 'Nvim'
|
||||||
msg = alternative and ('%s, use %s instead.'):format(msg, alternative) or msg
|
msg = alternative and ('%s, use %s instead.'):format(msg, alternative) or msg
|
||||||
|
@ -132,11 +132,13 @@ describe('lua stdlib', function()
|
|||||||
-- vim.deprecate(name, alternative, version, plugin, backtrace)
|
-- vim.deprecate(name, alternative, version, plugin, backtrace)
|
||||||
eq(dedent[[
|
eq(dedent[[
|
||||||
foo.bar() is deprecated, use zub.wooo{ok=yay} instead. :help deprecated
|
foo.bar() is deprecated, use zub.wooo{ok=yay} instead. :help deprecated
|
||||||
This feature will be removed in Nvim version 2.17]],
|
This feature will be removed in Nvim version 0.10]],
|
||||||
exec_lua('return vim.deprecate(...)', 'foo.bar()', 'zub.wooo{ok=yay}', '2.17'))
|
exec_lua('return vim.deprecate(...)', 'foo.bar()', 'zub.wooo{ok=yay}', '0.10'))
|
||||||
-- Same message, skipped.
|
-- Same message, skipped.
|
||||||
eq(vim.NIL,
|
eq(vim.NIL,
|
||||||
exec_lua('return vim.deprecate(...)', 'foo.bar()', 'zub.wooo{ok=yay}', '2.17'))
|
exec_lua('return vim.deprecate(...)', 'foo.bar()', 'zub.wooo{ok=yay}', '0.10'))
|
||||||
|
-- Don't show error if not hard deprecated
|
||||||
|
eq(vim.NIL, exec_lua('return vim.deprecate(...)', 'foo.bar()', 'nil', '5000.0.0'))
|
||||||
-- When `plugin` is specified, don't show ":help deprecated". #22235
|
-- When `plugin` is specified, don't show ":help deprecated". #22235
|
||||||
eq(dedent[[
|
eq(dedent[[
|
||||||
foo.bar() is deprecated, use zub.wooo{ok=yay} instead.
|
foo.bar() is deprecated, use zub.wooo{ok=yay} instead.
|
||||||
|
Loading…
Reference in New Issue
Block a user