Justin M. Keyes

Results 2414 comments of Justin M. Keyes

so let's move forward with gO but drop the other one for now.

> `gO` is used in `help` and `:Man` to display the outline, which is the meaning of `O`. However, our current `vim.lsp.buf.document_symbols` ignores the hierarchy and directly returns all symbols....

I will update the docs for `:help gO` in another PR.

Should `winnr()` only consider `:help focusable` windows?

> If non focus-able windows are counted, users may misunderstand that they can navigate through them with commands like `wincmd`. Yes, as documented at `:help focusable`, non-focusable windows are "special"...

Side-note: should we define and document "popups" as "unfocusable windows"? https://github.com/neovim/neovim/pull/34207#discussion_r2112045760

This sounds similar to https://github.com/neovim/nvim-lspconfig/pull/3515 You did not even mention which "root markers" are in these projects ?

after https://github.com/neovim/neovim/pull/33485 is merged, Nvim 0.12 / 0.11.2 will support more control over the precedence of `root_markers`. Then the go config can be updated (I'm hoping you or another user...

does this simplify anything in `vim.inspect_pos` ? https://github.com/neovim/neovim/blob/e049c6e4c08a141c94218672e770f86f91c27a11/runtime/lua/vim/_inspector.lua#L43 > Now the data returned by `get_captures_at_pos()` is: is that example supposed to have `id = ` fields?

Please try latest Nvim 0.11 from https://github.com/neovim/neovim/releases , using the `vim.lsp.config` feature. https://github.com/neovim/neovim/pull/31565 seems related, and should give a hint about where the relevant code lives. The "manager" part of...