correctly sending textDocument/didSave notifications

[prabirshrestha]

//cc @rcjsuen

The confusion started from rcjsuen/dockerfile-language-server#165.

I'm implementing the LSP server for vim at https://github.com/prabirshrestha/vim-lsp. And I would like to know when should it send textDocument/didSave changes to the client.

Here are the possibilities. (I would like to support both v2 and v3 porotocols).Is the table correct?

serverCapabilities	Send textDocument/didSave notification	includeText
{}	❌	❌
{ 'textDocumentSync: 0/1/2 }	✔️	❌
{ 'textDocumentSync: {} }	❌	❌
{ 'textDocumentSync: {save: {} } }	✔️	❌
{ 'textDocumentSync: { save: { includeText: true } } }	✔️	✔️
{ 'textDocumentSync: { save: { includeText: false} } }	✔️	❌

[dbaeumer]

@prabirshrestha your table is correct.

Please note that v1 didn't have a didSave notification. But you are not targeting this.