search

lukas-reineke / lsp-format.nvim

A wrapper around Neovims native LSP formatting.
559 stars 27 forks source link

Move the client support check in the format function #88

Closed GaetanLepage closed 1 day ago

GaetanLepage commented 2 months ago

Follow up of #87. This is an alternative way of getting around the dynamic registrations issue. We only ensure that the LSP client supports formatting in the format function.

GaetanLepage commented 2 months ago

@lukas-reineke is that the solution you were thinking about ?

lukas-reineke commented 2 months ago

Yes, basically :+1: Sorry for the delay, give me a couple of days. I need to find some time to test this and see if I want to change anything else.

GaetanLepage commented 2 months ago

Yes, basically 👍 Sorry for the delay, give me a couple of days. I need to find some time to test this and see if I want to change anything else.

Sure !

lukas-reineke commented 1 day ago

give me a couple of days.

... took a bit longer, sorry

GaetanLepage commented 1 day ago

... took a bit longer, sorry

No worries ! Thanks :)