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

[IMPORTANT] Deprecation & Transition Strategy

Open tjdevries opened this issue 5 years ago • 14 comments

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

  • PrevDiagnosticCycle and NextDiagnosticCycle:
    • New methods:
      • vim.lsp.diagnostic.goto_prev()
      • vim.lsp.diagnostic.goto_next()
    • Example Mappings:
      • nnoremap <leader>dn <cmd>lua vim.lsp.diagnostic.goto_next()<CR>
  • PrevDiagnostic and NextDiagnostic
    • New methods:
      • vim.lsp.diagnostic.goto_prev { wrap = false }
      • vim.lsp.diagnostic.goto_next { wrap = false }
    • Example Mappings:
      • nnoremap <leader>dn <cmd>lua vim.lsp.diagnostic.goto_next { wrap = false }<CR>
  • OpenDiagnostic
    • New method:
      • vim.lsp.diagnostic.set_loclist()

Configuring LSP Diagnostic Display

  • Configuration of the LSP diagnostic display is now done with lsp-handlers.
    • You can read more about them by doing :help lsp-handler
    • For example, to disable virtual text, you would do this in your init.vim
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

  • To configure more advanced usage, first, you should read the new help for ":help vim.lsp.diagnostic.on_publish_diagnostics"
  • After reading, you can override diagnostic-nvim configuration values, like sign configuration and virtual text spacing using something similar to the following:
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,
  }
)

tjdevries avatar Nov 06 '20 16:11 tjdevries

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

Scupake avatar Nov 29 '20 13:11 Scupake

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()

sentriz avatar Nov 29 '20 14:11 sentriz

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

Oh, hey that worked. Thanks!

Scupake avatar Nov 29 '20 14:11 Scupake

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

lucax88x avatar Dec 09 '20 17:12 lucax88x

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=

tjdevries avatar Dec 09 '20 17:12 tjdevries

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 avatar Dec 09 '20 18:12 lucax88x

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

Is this todo still a thing?

lucax88x avatar Dec 10 '20 09:12 lucax88x

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)

tjdevries avatar Dec 10 '20 20:12 tjdevries

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)

lucax88x avatar Dec 11 '20 08:12 lucax88x

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

tjdevries avatar Dec 13 '20 01:12 tjdevries

Wow, I am truly sorry.

LspDiagnosticsErrorSign => LspDiagnosticsSignError

I'm gonna hide behind a mountain.

lucax88x avatar Dec 14 '20 08:12 lucax88x

: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 :)

tjdevries avatar Dec 15 '20 21:12 tjdevries

Might wanna archive this repo, maybe.

runiq avatar May 21 '21 17:05 runiq

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

focusable=false is very useful

wllenyj avatar Jul 05 '21 15:07 wllenyj