nvim-plugins/telescope.nvim
1
0
Fork 0
mirror of https://github.com/nvim-telescope/telescope.nvim synced 2024-09-16 21:24:02 +02:00
Code Issues Projects Releases Packages Wiki Activity

fix(layout_strat): only validate strategy specific config values

Browse source
This commit is contained in:
James Trew 2024-01-24 22:22:50 -05:00
parent 36dce6261a
commit 8a14f24cca
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
lua/telescope/pickers/layout_strategies.lua
View file

@ -113,11 +113,11 @@ local adjust_pos = function(pos, ...)
end
end
--@param strategy_name string: the name of the layout_strategy we are validating for
--@param configuration table: table with keys for each option available
--@param values table: table containing all of the non-default options we want to set
--@param default_layout_config table: table with the default values to configure layouts
--@return table: table containing the combined options (defaults and non-defaults)
---@param strategy_name string: the name of the layout_strategy we are validating for
---@param configuration table: table with keys for each option available
---@param values table: table containing all of the non-default options we want to set
---@param default_layout_config table: table with the default values to configure layouts
---@return table: table containing the combined options (defaults and non-defaults)
local function validate_layout_config(strategy_name, configuration, values, default_layout_config)
assert(strategy_name, "It is required to have a strategy name for validation.")
local valid_configuration_keys = get_valid_configuration_keys(configuration)
@ -159,7 +159,7 @@ local function validate_layout_config(strategy_name, configuration, values, defa
end
-- Always set the values passed first.
for k in pairs(values) do
for k in pairs(values[strategy_name]) do
if not valid_configuration_keys[k] then
-- TODO: At some point we'll move to error here,
-- but it's a bit annoying to just straight up crash everyone's stuff.