nvimtools / none-ls.nvim

null-ls.nvim reloaded / Use Neovim as a language server to inject LSP diagnostics, code actions, and more via Lua.
The Unlicense
2.39k stars 69 forks source link

Bug: None-ls clients not loading anymore after the recent commits #88

Closed Zeioth closed 5 months ago

Zeioth commented 5 months ago

FAQ

Issues

Neovim Version

v0.9.5

Dev Version?

Operating System

arch

Minimal Config

x

Steps to Reproduce

x

Reproducibility Check

Expected Behavior

x

Actual Behavior

x

Debug Log

x

Help

No

Implementation Help

After the commit 92a0251280ae2cce790ce5733e223459a57567d0 my previous config has stopped to work.

    opts = function()
      local nls = require "null-ls"
      return {
        sources = {
          -- You can customize your formatters here.
          nls.builtins.formatting.beautysh.with {
            command = "beautysh",
            args = { "--indent-size=2", "$FILENAME" },
          },
          -- TODO: Disable the next feature once this has been merged.
          -- https://github.com/bash-lsp/bash-language-server/issues/933
          nls.builtins.code_actions.shellcheck,
          nls.builtins.diagnostics.shellcheck.with { diagnostics_format = "" },
        },
        on_attach = utils_lsp.apply_user_lsp_settings,
      }

What's the new way to do this?

Requirements

Zeioth commented 5 months ago

screenshot_2024-03-06_01-16-15_860951787

Zeioth commented 5 months ago

If I comment the lines above, I don't get the error anymore, but still none-ls doesn't seem to load its clients normally anymore (another bug? Same one?). screenshot_2024-03-06_01-17-09_313894963

Normally they load automatically. But now If I run NullLsInfo I get this: screenshot_2024-03-06_01-18-33_936655183

I'm updating to main HEAD from d991c8952defd654ed78cce0abce63a5c06d0bc6, commit where everything was working OK.

Zeioth commented 5 months ago

Just to clarify, I have installed shellcheck and beautysh using mason + mason-null-ls.nvim, so the clients should be automatically loaded, same as always.

mochaaP commented 5 months ago

Duplicate of #81, search beautysh in the thread.