nvf icon indicating copy to clipboard operation
nvf copied to clipboard

vim.lsp.enable not enabling LSP

Open staticdev opened this issue 7 months ago • 4 comments

⚠️ Please verify that this bug has NOT been reported before.

  • [x] I checked all existing issues and didn't find a similar issue

Description

I believe this option is not the correct to enable LSP since I saw also vim.languages.enableLSP also work, but maybe the correct behavior is this one to alias the other config since it is misleading users to believe it will enable the languages.

👟 Reproduction steps

  programs.nvf = {
    enable = true;
    settings = {
      vim = {
        lsp.enable = true;
        languages = {
          nix.enable = true;
        };
        statusline.lualine.enable = true;
      };
   };

👀 Expected behavior

Show that LSP is enabled.

😓 Actual Behavior

Image

💻 Metadata

📝 Relevant log output

-

staticdev avatar Apr 25 '25 10:04 staticdev

vim.languages.enableLSP = true;

This option might do the trick for you

adxthya avatar Apr 26 '25 22:04 adxthya

I should probably set enableLSP to true when vim.lsp.enable = true;

NotAShelf avatar Apr 26 '25 22:04 NotAShelf

I should probably set enableLSP to true when vim.lsp.enable = true;

I guess so. They are written absolutely the same, I still did not get the difference. If you auto set this, would make user experience better for sure.

I would even make both enabled if any language is enabled, do they have any use if LSP is off?

staticdev avatar Apr 27 '25 02:04 staticdev

Addressed in c2620716fdc95f2e6a024ee2595f164dc8f6a310, which should be in the main branch soon.

NotAShelf avatar Apr 27 '25 03:04 NotAShelf