search

theHamsta / nvim-dap-virtual-text

GNU General Public License v3.0
889 stars 24 forks source link

require('nvim-dap-virtual-text').setup() has not enough argument according to LIPS LSP #65

Open stephane-archer opened 1 year ago

stephane-archer commented 1 year ago

my lips LPS complain about the following line not receiving enough arguments: require('nvim-dap-virtual-text').setup() I imagine the solution would be require('nvim-dap-virtual-text').setup({}) but I'm not sure, I'm really a noob when it comes to configuring Neovim. Please update the README

theHamsta commented 11 months ago

This seems to be a bug in https://github.com/LuaLS/lua-language-server. It seems to count the number of arguments here https://github.com/theHamsta/nvim-dap-virtual-text/blob/99f0a58f96a44198267d75e613b732c9b6f93845/lua/nvim-dap-virtual-text.lua#L78-L91 incorrectly because of this line. When I remove https://github.com/theHamsta/nvim-dap-virtual-text/blob/99f0a58f96a44198267d75e613b732c9b6f93845/lua/nvim-dap-virtual-text.lua#L84 then it recognizes correctly that the function has only 5 parameters and 1 return value.

Not sure why I can't repro it anymore. Maybe it has been fixed upstream?