tree dc3269497ed649a8bbb910bd6cd1360ba82484cd
parent 582c62ec74d06936c88d4b760c63cbc2925e69c7
author Rebecca Stambler <rstambler@golang.org> 1602564648 -0400
committer Rebecca Stambler <rstambler@golang.org> 1604278673 +0000

internal/lsp: change `go mod vendor` warning into a diagnostic

This CL uses the approach of a source.ErrorList for the `go mod vendor`
error--rather than a ShowMessageRequest. The suggested fix is a command
that runs `go mod vendor`, so I added a CommandFix to the source.Error
type.

I'm not sure if a diagnostic is better than a ShowMessageRequest--
perhaps it should be both?

Fixes golang/go#41819
Fixes golang/go#42152

Change-Id: I4ba2d7af0233f13583395e871166caed83454d6e
Reviewed-on: https://go-review.googlesource.com/c/tools/+/261737
Trust: Rebecca Stambler <rstambler@golang.org>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>