golang/go

x/tools/gopls: support LocationLink for relevant requests

Aperta

#40.195 aperta il 13 lug 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
FeatureRequestToolsgoplshelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

When requesting textDocument/definition gopls returns a Location.

This FR is about supporting returning a LocationLink instead for clients that support it (a client capability).

Rationale: some editors/plugins can use this to support previewing the entire function on hover. See https://github.com/neovim/neovim/issues/12218 and https://github.com/neovim/neovim/pull/12368 for an in-depth description.

In terms of other language servers that do this, it looks like rust-analyzer does it as well.

All (current) LSP actions that can return LocationLink's:

  • textDocument/declaration
  • textDocument/definition
  • textDocument/typeDefinition

Guida contributor

Direzione di ricerca
Implementa il supporto per LocationLink in gopls. Modifica definition.go e i gestori correlati per controllare le capacità del client per LocationLink e restituire LocationLink invece di Location quando supportato. Studia la specifica LSP per LocationLink e le implementazioni esistenti come rust analyzer.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoLSP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30