Gregory Anders

We can wait to merge as long as you need.

> Can `root_markers` optionally be a function that takes in the cwd and returns the root directory? The `root_dir` option can do this. It accepts either a string or a...

`vim.lsp.config()` (with no arguments) returns a deepcopy of the full configuration table.

>Just a drive by comment on my concern over the inclusion of the iterm2 protocol I share this concern. The iTerm2 protocol is supported by only 2 terminals: iTerm2 and...

>This PR fixes this by handling regtype properly. How does it do this though? It looks like this only works if the text that is being pasted already contains the...

Can you please provide a test case that **fails** before your change and **passes** after implementing your change? By the way, it is fine if are using an LLM to...

>The failing scenario (working after the change) is in the description of this MR. The test needs to be an automated test added to the test suite. >The extra line...

> @gpanders I just implemented the cache solution similar to what is done in [clipboard.vim](https://github.com/neovim/neovim/blob/master/runtime/autoload/provider/clipboard.vim#L281-L283) > > * no more dirty new line with the regtype, no side effect >...

Requiring an explicit revision for move/set makes sense to me since those commands can be more ambiguous. However, requiring a revision for `bookmark create` adds a bit of friction which...

It’s not an implementation detail. In the old days, diagnostics and LSP were intertwined. It was messy and there was no clean way to add diagnostics from non-LSP sources. Dedicated...