ojroques / nvim-lspfuzzy

A Neovim plugin to make the LSP client use FZF
BSD 2-Clause "Simplified" License
316 stars 11 forks source link

bad argument #1 to 'unpack' (table expected, got string) #7

Closed teto closed 3 years ago

teto commented 3 years ago

I've updated everything and I am on neovim master but I get

|| E5108: Error executing lua ...to/.local/share/nvim/site/nvim-lspfuzzy/lua/lspfuzzy.lua:95: bad argument #1 to 'unpack' (table expected, got string)
|| E5108: Error executing lua ...to/.local/share/nvim/site/nvim-lspfuzzy/lua/lspfuzzy.lua:95: bad argument #1 to 'unpack' (table expected, got string)

on :LspDiagnosticAll

ojroques commented 3 years ago

Hello!

I'll look into it. Does that happen only for :LspDiagnosticAll? Are you using the fzf_preview_window option in your config?

teto commented 3 years ago

Nope it fails for LspDiagnostic too, if I comment out let g:fzf_preview_window = 'right:30%', then lspfuzzy works

ojroques commented 3 years ago

Should work now. The plugin was expecting a table for that option. I added a check to convert a string value to a table.