Add LspDiag nextWrap and prevWrap command args

yegappan / lsp

Language Server Protocol (LSP) plugin for Vim9

MIT License

447 stars 43 forks source link

Add LspDiag nextWrap and prevWrap command args #482

Closed nickspoons closed 2 months ago

nickspoons commented 4 months ago

Add new 'nextWrap' and 'prevWrap' arguments for the :LspDiag command, allowing wrapping around to the first/last diagnostics instead of displaying the "No more diagnostics found" message.

Closes #413

yegappan commented 3 months ago

Thanks for the patch. Can you fix the following test failure?

command line..script /home/runner/work/lsp/lsp/test/runner.vim[48]..function <SNR>1_LspRunTests[20]..Test_LspDiagsSubcmd
line 4: Expected 'LspDiag first current here highlight last next prev show' but got 'LspDiag first current here highlight last next nextWrap prev prevWrap show'
Test_LspDiagsSubcmd(): FAIL

nickspoons commented 3 months ago

Hi @yegappan, I've fixed the test by adding the missing args to the assertion.