vim-lsp-ale icon indicating copy to clipboard operation
vim-lsp-ale copied to clipboard

notify_diag_to_ale not incrementing idx on continue

Open alexkornitzer opened this issue 3 years ago • 3 comments

Hey,

Happy to file a PR if it helps but notify_diag_to_ale is not extracting all the results correctly as idx is not incremented when the entry is skipped due to threshold checks and thus keeps analysing the same entry.

https://github.com/rhysd/vim-lsp-ale/blob/master/autoload/lsp/ale.vim#L114-L116

alexkornitzer avatar Aug 01 '22 17:08 alexkornitzer

@alexkornitzer Hi, would you mind sharing the fix or point me to a tree which carries the patch? Thanks!

plaimbock avatar Sep 08 '24 11:09 plaimbock

I moved to neovim a while back but iirc you just need to move line 120 to before line 114 so the idx increments before the continue while not impacting the indexing at 113.

alexkornitzer avatar Sep 10 '24 09:09 alexkornitzer

I moved to neovim a while back but iirc you just need to move line 120 to before line 114 so the idx increments before the continue while not impacting the indexing at 113.

Thanks!

plaimbock avatar Sep 10 '24 12:09 plaimbock