doom-nvim icon indicating copy to clipboard operation
doom-nvim copied to clipboard

[BUG][next] luacheck required?

Open baurmatt opened this issue 3 years ago • 3 comments

Current Behavior:

[null-ls] [ERROR 16:17:39] ...t/null-ls.nvim/lua/null-ls/helpers/generator_factory.lua:180: command luacheck is not executable (make sure it's installed and on your $PATH)

Expected Behavior:

No error pops up :) The README doesn't mention luacheck as a dependency :)

Steps To Reproduce:

Logs

Check Health Output
Doom Report Output

Anything else:

baurmatt avatar Jul 18 '22 14:07 baurmatt

Null-ls is a bit of a challenge because there's no auto-installer for it so it relies on system dependencies so if you want it to work you'll have to install it luarocks install luacheck. What's a good way to make it clearer to the user that it needs to be installed? I could at a note in the dependencies section of the readme.

Also thanks for these bugreports :) They don't seem to hard so I'll sort them out soon hopefully.

connorgmeehan avatar Jul 18 '22 14:07 connorgmeehan

Ah, so this error popups up because lsp is now enabled by default for all languages unlike in the current stable where lsp is disable for lua by default? Does this affect other languages as well?

Can we disable it by default for lua? ;)

baurmatt avatar Jul 18 '22 14:07 baurmatt

Ok now that https://github.com/williamboman/mason.nvim has been released this is something we'll implement for the next release.

connorgmeehan avatar Jul 29 '22 04:07 connorgmeehan