fix(init): migrate vim.validate to positional parameter API (#154)
## Problem
Neovim 0.11+ deprecated the table-based `vim.validate({...})` form.
Every
plugin load produces 5 deprecation warnings pointing at `init.lua`
config
validation.
## Solution
Convert all `vim.validate` calls in `init()` to the new positional
`vim.validate(name, value, validator, optional_or_msg)` form. No
behavioral
change — identical validation logic, just the calling convention.
This commit is contained in:
Barrett Ruth
GitHub
parent
58589947e8
commit
c498fd2bac
1 changed files with 85 additions and 102 deletions
|
|
@ -591,139 +591,122 @@ local function init()
|
||||||
opts.neogit = {}
|
opts.neogit = {}
|
||||||
end
|
end
|
||||||
|
|
||||||
vim.validate({
|
vim.validate('debug', opts.debug, function(v)
|
||||||
debug = {
|
return v == nil or type(v) == 'boolean' or type(v) == 'string'
|
||||||
opts.debug,
|
end, 'boolean or string (file path)')
|
||||||
function(v)
|
vim.validate('hide_prefix', opts.hide_prefix, 'boolean', true)
|
||||||
return v == nil or type(v) == 'boolean' or type(v) == 'string'
|
vim.validate('fugitive', opts.fugitive, function(v)
|
||||||
end,
|
return v == nil or v == false or type(v) == 'table'
|
||||||
'boolean or string (file path)',
|
end, 'table or false')
|
||||||
},
|
vim.validate('neogit', opts.neogit, function(v)
|
||||||
hide_prefix = { opts.hide_prefix, 'boolean', true },
|
return v == nil or v == false or type(v) == 'table'
|
||||||
fugitive = {
|
end, 'table or false')
|
||||||
opts.fugitive,
|
vim.validate('extra_filetypes', opts.extra_filetypes, 'table', true)
|
||||||
function(v)
|
vim.validate('highlights', opts.highlights, 'table', true)
|
||||||
return v == nil or v == false or type(v) == 'table'
|
|
||||||
end,
|
|
||||||
'table or false',
|
|
||||||
},
|
|
||||||
neogit = {
|
|
||||||
opts.neogit,
|
|
||||||
function(v)
|
|
||||||
return v == nil or v == false or type(v) == 'table'
|
|
||||||
end,
|
|
||||||
'table or false',
|
|
||||||
},
|
|
||||||
extra_filetypes = { opts.extra_filetypes, 'table', true },
|
|
||||||
highlights = { opts.highlights, 'table', true },
|
|
||||||
})
|
|
||||||
|
|
||||||
if opts.highlights then
|
if opts.highlights then
|
||||||
vim.validate({
|
vim.validate('highlights.background', opts.highlights.background, 'boolean', true)
|
||||||
['highlights.background'] = { opts.highlights.background, 'boolean', true },
|
vim.validate('highlights.gutter', opts.highlights.gutter, 'boolean', true)
|
||||||
['highlights.gutter'] = { opts.highlights.gutter, 'boolean', true },
|
vim.validate('highlights.blend_alpha', opts.highlights.blend_alpha, 'number', true)
|
||||||
['highlights.blend_alpha'] = { opts.highlights.blend_alpha, 'number', true },
|
vim.validate('highlights.overrides', opts.highlights.overrides, 'table', true)
|
||||||
['highlights.overrides'] = { opts.highlights.overrides, 'table', true },
|
vim.validate('highlights.context', opts.highlights.context, 'table', true)
|
||||||
['highlights.context'] = { opts.highlights.context, 'table', true },
|
vim.validate('highlights.treesitter', opts.highlights.treesitter, 'table', true)
|
||||||
['highlights.treesitter'] = { opts.highlights.treesitter, 'table', true },
|
vim.validate('highlights.vim', opts.highlights.vim, 'table', true)
|
||||||
['highlights.vim'] = { opts.highlights.vim, 'table', true },
|
vim.validate('highlights.intra', opts.highlights.intra, 'table', true)
|
||||||
['highlights.intra'] = { opts.highlights.intra, 'table', true },
|
vim.validate('highlights.priorities', opts.highlights.priorities, 'table', true)
|
||||||
['highlights.priorities'] = { opts.highlights.priorities, 'table', true },
|
|
||||||
})
|
|
||||||
|
|
||||||
if opts.highlights.context then
|
if opts.highlights.context then
|
||||||
vim.validate({
|
vim.validate('highlights.context.enabled', opts.highlights.context.enabled, 'boolean', true)
|
||||||
['highlights.context.enabled'] = { opts.highlights.context.enabled, 'boolean', true },
|
vim.validate('highlights.context.lines', opts.highlights.context.lines, 'number', true)
|
||||||
['highlights.context.lines'] = { opts.highlights.context.lines, 'number', true },
|
|
||||||
})
|
|
||||||
end
|
end
|
||||||
|
|
||||||
if opts.highlights.treesitter then
|
if opts.highlights.treesitter then
|
||||||
vim.validate({
|
vim.validate(
|
||||||
['highlights.treesitter.enabled'] = { opts.highlights.treesitter.enabled, 'boolean', true },
|
'highlights.treesitter.enabled',
|
||||||
['highlights.treesitter.max_lines'] = {
|
opts.highlights.treesitter.enabled,
|
||||||
opts.highlights.treesitter.max_lines,
|
'boolean',
|
||||||
'number',
|
true
|
||||||
true,
|
)
|
||||||
},
|
vim.validate(
|
||||||
})
|
'highlights.treesitter.max_lines',
|
||||||
|
opts.highlights.treesitter.max_lines,
|
||||||
|
'number',
|
||||||
|
true
|
||||||
|
)
|
||||||
end
|
end
|
||||||
|
|
||||||
if opts.highlights.vim then
|
if opts.highlights.vim then
|
||||||
vim.validate({
|
vim.validate('highlights.vim.enabled', opts.highlights.vim.enabled, 'boolean', true)
|
||||||
['highlights.vim.enabled'] = { opts.highlights.vim.enabled, 'boolean', true },
|
vim.validate('highlights.vim.max_lines', opts.highlights.vim.max_lines, 'number', true)
|
||||||
['highlights.vim.max_lines'] = { opts.highlights.vim.max_lines, 'number', true },
|
|
||||||
})
|
|
||||||
end
|
end
|
||||||
|
|
||||||
if opts.highlights.intra then
|
if opts.highlights.intra then
|
||||||
vim.validate({
|
vim.validate('highlights.intra.enabled', opts.highlights.intra.enabled, 'boolean', true)
|
||||||
['highlights.intra.enabled'] = { opts.highlights.intra.enabled, 'boolean', true },
|
vim.validate('highlights.intra.algorithm', opts.highlights.intra.algorithm, function(v)
|
||||||
['highlights.intra.algorithm'] = {
|
return v == nil or v == 'default' or v == 'vscode'
|
||||||
opts.highlights.intra.algorithm,
|
end, "'default' or 'vscode'")
|
||||||
function(v)
|
vim.validate('highlights.intra.max_lines', opts.highlights.intra.max_lines, 'number', true)
|
||||||
return v == nil or v == 'default' or v == 'vscode'
|
|
||||||
end,
|
|
||||||
"'default' or 'vscode'",
|
|
||||||
},
|
|
||||||
['highlights.intra.max_lines'] = { opts.highlights.intra.max_lines, 'number', true },
|
|
||||||
})
|
|
||||||
end
|
end
|
||||||
|
|
||||||
if opts.highlights.priorities then
|
if opts.highlights.priorities then
|
||||||
vim.validate({
|
vim.validate('highlights.priorities.clear', opts.highlights.priorities.clear, 'number', true)
|
||||||
['highlights.priorities.clear'] = { opts.highlights.priorities.clear, 'number', true },
|
vim.validate(
|
||||||
['highlights.priorities.syntax'] = { opts.highlights.priorities.syntax, 'number', true },
|
'highlights.priorities.syntax',
|
||||||
['highlights.priorities.line_bg'] = { opts.highlights.priorities.line_bg, 'number', true },
|
opts.highlights.priorities.syntax,
|
||||||
['highlights.priorities.char_bg'] = { opts.highlights.priorities.char_bg, 'number', true },
|
'number',
|
||||||
})
|
true
|
||||||
|
)
|
||||||
|
vim.validate(
|
||||||
|
'highlights.priorities.line_bg',
|
||||||
|
opts.highlights.priorities.line_bg,
|
||||||
|
'number',
|
||||||
|
true
|
||||||
|
)
|
||||||
|
vim.validate(
|
||||||
|
'highlights.priorities.char_bg',
|
||||||
|
opts.highlights.priorities.char_bg,
|
||||||
|
'number',
|
||||||
|
true
|
||||||
|
)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
if type(opts.fugitive) == 'table' then
|
if type(opts.fugitive) == 'table' then
|
||||||
---@type diffs.FugitiveConfig
|
---@type diffs.FugitiveConfig
|
||||||
local fug = opts.fugitive
|
local fug = opts.fugitive
|
||||||
vim.validate({
|
vim.validate('fugitive.horizontal', fug.horizontal, function(v)
|
||||||
['fugitive.horizontal'] = {
|
return v == nil or v == false or type(v) == 'string'
|
||||||
fug.horizontal,
|
end, 'string or false')
|
||||||
function(v)
|
vim.validate('fugitive.vertical', fug.vertical, function(v)
|
||||||
return v == nil or v == false or type(v) == 'string'
|
return v == nil or v == false or type(v) == 'string'
|
||||||
end,
|
end, 'string or false')
|
||||||
'string or false',
|
|
||||||
},
|
|
||||||
['fugitive.vertical'] = {
|
|
||||||
fug.vertical,
|
|
||||||
function(v)
|
|
||||||
return v == nil or v == false or type(v) == 'string'
|
|
||||||
end,
|
|
||||||
'string or false',
|
|
||||||
},
|
|
||||||
})
|
|
||||||
end
|
end
|
||||||
|
|
||||||
if opts.conflict then
|
if opts.conflict then
|
||||||
vim.validate({
|
vim.validate('conflict.enabled', opts.conflict.enabled, 'boolean', true)
|
||||||
['conflict.enabled'] = { opts.conflict.enabled, 'boolean', true },
|
vim.validate('conflict.disable_diagnostics', opts.conflict.disable_diagnostics, 'boolean', true)
|
||||||
['conflict.disable_diagnostics'] = { opts.conflict.disable_diagnostics, 'boolean', true },
|
vim.validate('conflict.show_virtual_text', opts.conflict.show_virtual_text, 'boolean', true)
|
||||||
['conflict.show_virtual_text'] = { opts.conflict.show_virtual_text, 'boolean', true },
|
vim.validate(
|
||||||
['conflict.format_virtual_text'] = { opts.conflict.format_virtual_text, 'function', true },
|
'conflict.format_virtual_text',
|
||||||
['conflict.show_actions'] = { opts.conflict.show_actions, 'boolean', true },
|
opts.conflict.format_virtual_text,
|
||||||
['conflict.priority'] = { opts.conflict.priority, 'number', true },
|
'function',
|
||||||
['conflict.keymaps'] = { opts.conflict.keymaps, 'table', true },
|
true
|
||||||
})
|
)
|
||||||
|
vim.validate('conflict.show_actions', opts.conflict.show_actions, 'boolean', true)
|
||||||
|
vim.validate('conflict.priority', opts.conflict.priority, 'number', true)
|
||||||
|
vim.validate('conflict.keymaps', opts.conflict.keymaps, 'table', true)
|
||||||
|
|
||||||
if opts.conflict.keymaps then
|
if opts.conflict.keymaps then
|
||||||
local keymap_validator = function(v)
|
local keymap_validator = function(v)
|
||||||
return v == false or type(v) == 'string'
|
return v == false or type(v) == 'string'
|
||||||
end
|
end
|
||||||
for _, key in ipairs({ 'ours', 'theirs', 'both', 'none', 'next', 'prev' }) do
|
for _, key in ipairs({ 'ours', 'theirs', 'both', 'none', 'next', 'prev' }) do
|
||||||
vim.validate({
|
vim.validate(
|
||||||
['conflict.keymaps.' .. key] = {
|
'conflict.keymaps.' .. key,
|
||||||
opts.conflict.keymaps[key],
|
opts.conflict.keymaps[key],
|
||||||
keymap_validator,
|
keymap_validator,
|
||||||
'string or false',
|
'string or false'
|
||||||
},
|
)
|
||||||
})
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue