microsoft/language-server-protocol

DidChangeConfigurationParams should contain a workspace URI

Open

#457 aperta il 22 apr 2018

Vedi su GitHub
 (1 commento) (2 reazioni) (0 assegnatari)HTML (956 fork)batch import
configurationfeature-requesthelp wanted

Metriche repository

Star
 (12.810 star)
Metriche merge PR
 (Merge medio 19h 29m) (12 PR mergiate in 30 g)

Descrizione

With 3.6.0 the protocol specification got multi-workspace support, however the workspace/didChangeConfiguration notification is still specced for single workspace roots. Often there is project/workspace specific configuration which should also trigger the workspace/didChangeConfiguration notification, however the server has no way of knowing for which workspace this new configuration applies and could just apply it to all workspaces which may yield in wrong future behaviour of the server.

Adding the workspace URI to DidChangeConfigurationParams is an easy way to make servers aware of workspace specific configuration without breaking backwards compatibility. If the field is unset/empty the server can just treat it like it did before such a change and if the server doesn't even support it, it will just treat it like before.

Guida contributor

Direzione di ricerca
Studia la definizione esistente di `DidChangeConfigurationParams` nella specifica LSP, esamina come altre notifiche gestiscono gli URI dello spazio di lavoro (es. `didChangeWatchedFiles`), quindi aggiungi un campo opzionale `workspaceUri` ai parametri. Invia una pull request con la modifica.
Tech stack
Nessuno
Dominio
api
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80