nvim-lua / diagnostic-nvim

A wrapper for neovim built in LSP diagnosis config
Apache License 2.0
219 stars 15 forks source link

[IMPORTANT] Deprecation & Transition Strategy #73

Open tjdevries opened 3 years ago

tjdevries commented 3 years ago

Now that https://github.com/neovim/neovim/pull/12655 is merged, we are deprecating diagnostic-nvim. For full information and commit message, see: https://github.com/neovim/neovim/commit/f75be5e9d510d5369c572cf98e78d9480df3b0bb

Any features that were in diagnostic-nvim have now been implemented in Neovim core! This was always the plan for diagnostic-nvim, as it was a playground and testing area for features and interfaces for the builtin LSP

You should remove any on_attach calls from diagnostic-nvim in your configuration. They are no longer required.

Common Actions

Configuring LSP Diagnostic Display

lua << EOF
vim.lsp.handlers["textDocument/publishDiagnostics"] = vim.lsp.with(
  vim.lsp.diagnostic.on_publish_diagnostics, {
    -- This will disable virtual text, like doing:
    -- let g:diagnostic_enable_virtual_text = 0
    virtual_text = false,

    -- This is similar to:
    -- let g:diagnostic_show_sign = 1
    -- To configure sign display,
    --  see: ":help vim.lsp.diagnostic.set_signs()"
    signs = true,

    -- This is similar to:
    -- "let g:diagnostic_insert_delay = 1"
    update_in_insert = false,
  }
)
EOF

Also note, the highlight group names have changed to now be consistent with each other. From the commit message in neovim core:

- Changed the highlight groups for LspDiagnostic highlight as they were
  inconsistently named.
    - For more information, see |lsp-highlight-diagnostics|

For example, the highlight that was formerly LspDiagnosticsError is now LspDiagnosticsVirtualTextError. It can also be configured by changing the default highlight group, LspDiagnosticsDefaultError. For more information, read :help lsp-highlight-diagnostics.

Advanced configuration

vim.lsp.handlers["textDocument/publishDiagnostics"] = vim.lsp.with(
  vim.lsp.diagnostic.on_publish_diagnostics, {
    -- Enable underline, use default values
    underline = true,
    -- Enable virtual text, override spacing to 4
    virtual_text = {
      spacing = 4,
      prefix = '~',
    },
    -- Use a function to dynamically turn signs off
    -- and on, using buffer local variables
    signs = function(bufnr, client_id)
      local ok, result = pcall(vim.api.nvim_buf_get_var, bufnr, 'show_signs')
      -- No buffer local variable set, so just enable by default
      if not ok then
        return true
      end

      return result
    end,
    -- Disable a feature
    update_in_insert = false,
  }
)
Scupake commented 3 years ago

Can someone please show me to how to do this autocmd CursorHold * lua vim.lsp.util.show_line_diagnostics() now?

sentriz commented 3 years ago

Can someone please show me to how to do this autocmd CursorHold * lua vim.lsp.util.show_line_diagnostics() now?

have you tried autocmd CursorHold * lua vim.lsp.diagnostic.show_line_diagnostics()

Scupake commented 3 years ago

have you tried autocmd CursorHold * lua vim.lsp.diagnostic.show_line_diagnostics()

Oh, hey that worked. Thanks!

lucax88x commented 3 years ago

Was someone able to change the signs icons? I tried several ways but nothing worked.

tjdevries commented 3 years ago

have you tried :help vim.lsp.diagnostics.set_signs() info?

    sign define LspDiagnosticsSignError text=E texthl=LspDiagnosticsSignError linehl= numhl=
    sign define LspDiagnosticsSignWarning text=W texthl=LspDiagnosticsSignWarning linehl= numhl=
    sign define LspDiagnosticsSignInformation text=I texthl=LspDiagnosticsSignInformation linehl= numhl=
    sign define LspDiagnosticsSignHint text=H texthl=LspDiagnosticsSignHint linehl= numhl=
lucax88x commented 3 years ago

Yep,

I tried this in the init.vim and all the possible lua versions

vim.api.nvim_call_function("sign_define", {"LspDiagnosticsErrorSign", {text = "", texthl = "LspDiagnosticsError"}}) vim.fn.sign_define("LspDiagnosticsErrorSign", {text = "", texthl = "LspDiagnosticsError"}) vim.cmd [[ sign define LspDiagnosticsErrorSign text= texthl=LspDiagnosticsError linehl= numhl= ]]

I tried them on_attach, in the new beforeDiagnostick callback, before and after any lsp setting, noone worked, to me.

lucax88x commented 3 years ago

https://github.com/neovim/neovim/blob/bfed67e00ecdf71e0c7d17b1fd802f223b42c800/test/functional/plugin/lsp/diagnostic_spec.lua#L700

Is this todo still a thing?

tjdevries commented 3 years ago

That todo is for a different thing (there was some error when trying to find the signs that had been displayed. it's an internal bug I think, not related or due to the code).

Have you tried it via the ex command, not using sign_define? I think there may be some bugs related to sign functions and commands (I do not know for sure though, I have not had time to investigate)

lucax88x commented 3 years ago

No worries, cosmetics is not as important as functionalities, and for now LSP is a great piece of software so I can definitely wait.

(truth is I don't know know what parameters to pass to :lua vim.lsp.diagnostic.set_signs, still a newb on lua :D)

tjdevries commented 3 years ago

Ah, I meant to just try the ex commands listed in the docs to configure the signs.

lucax88x commented 3 years ago

Wow, I am truly sorry.

LspDiagnosticsErrorSign => LspDiagnosticsSignError

I'm gonna hide behind a mountain.

tjdevries commented 3 years ago

:laughing: such is life. No worries. I changed the names in the PR because they were randomly named before and now follow a pattern, so at least you're not too crazy :)

runiq commented 3 years ago

Might wanna archive this repo, maybe.

wllenyj commented 3 years ago

autocmd CursorHold * lua vim.lsp.diagnostic.show_line_diagnostics{focusable=false}

focusable=false is very useful