feat: support vim.diagnostic.is_disabled()

[sstallion]

This PR adds support for checking vim.diagnostic.is_disabled() on nvim 0.9+. This function was added to support globally enabling or disabling diagnostics. This is especially useful for setups where diagnostics are enabled conditionally or on a project basis. There is no change in behavior for versions prior to 0.9.

Thanks for a great plugin!

[alex-courtis]

Works nicely. We don't get a DiagnosticChanged event until switching buffers or writing, but that's Good Enough: the tree is refreshed reasonably quickly.

Many thanks for your contribution.

[alex-courtis]

Please resolve styling issue.

[sstallion]

Done. Thanks again!

[sstallion]

I wonder if adding something similar to the following autocommands (from lsp.txt) would be enough:

autocmd User LspProgressUpdate redrawstatus
autocmd User LspRequest redrawstatus

I'll play around with this a bit today to see if this can close the gap.

[alex-courtis]

I'll play around with this a bit today to see if this can close the gap.

That would be great, thank you. Instant updates would look great.

[sstallion]

That would be great, thank you. Instant updates would look great.

I've been looking into this, this evening and it looks like updates may already be instant. After from_nvim_lsp() is called, signs will be cleared, which should achieve the desired result.

Would it be okay to merge this PR as-is? I'm not seeing much else that can be done to improve the responsiveness at this point.

[alex-courtis]

Would it be okay to merge this PR as-is? I'm not seeing much else that can be done to improve the responsiveness at this point.

That's fine. The UX is great.

[alex-courtis]

Many thanks for your contribution.

[sstallion]

My pleasure. Thanks for a great plugin!