CL 643280 changed the behavior of debug adapter in remote mode from
always "legacy" to the result from command
`dlv substitute-path-guess-helper`.

This CL calls the method to find the right return value and test the
debug adapter with the command return.

In master branch, the integration test is running against preview
version of extension, so the debug adapter is always "dlv-dap"
regardless of the command return. So this test is not captured
in the master branch.

Change-Id: I5ce6e1040fa16f10b038265d7e0ab7a884a99141
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/684475
kokoro-CI: kokoro <[email protected]>
Reviewed-by: Robert Findley <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
(cherry picked from commit 09dcada)
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/684018

This is an automated CL which updates the CHANGELOG.md.

Change-Id: I8848b7353295d9463286e9d249a23ac47dd7c44b
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/684855
kokoro-CI: kokoro <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Gopher Robot <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>

This is an automated CL which updates the package.json and package-lock.json.

Change-Id: I5c14653fab4d0f1f1bf00542b543fc4ed721fade
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/685015
kokoro-CI: kokoro <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Gopher Robot <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
Reviewed-by: Robert Findley <[email protected]>

This is an automated CL which updates the CHANGELOG.md.

Change-Id: I7eccb56eb23777f4086c384e79ed3a3799beff43
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/685975
kokoro-CI: kokoro <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
Reviewed-by: Robert Findley <[email protected]>
Auto-Submit: Gopher Robot <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

This is an automated CL which updates the gopls version and settings.

For golang/go#74655

Change-Id: I4e99b890e321533d29a781b7554074452cb22cd0
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/688555
kokoro-CI: kokoro <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Alan Donovan <[email protected]>
Auto-Submit: Gopher Robot <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
Reviewed-by: Alan Donovan <[email protected]>

This is an automated CL which updates the gopls version and settings.

For golang/go#74655

Change-Id: Ia680c5d07c698c8ae539b1aa9923d390ae1d3736
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/691035
Auto-Submit: Gopher Robot <[email protected]>
kokoro-CI: kokoro <[email protected]>
Reviewed-by: Alan Donovan <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

CL 559738 changes the type of output channel to log output channel.
The log output channel does not show by default.

The extension will only show the output channel upon failure, show
information message upon success.

For #3359

Change-Id: Id4bb8ff053748c80ac3192db34aed0c4c9c5acec
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/691615
Auto-Submit: Hongxiang Jiang <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>
kokoro-CI: kokoro <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

The info() is the implementation of append().

The append() is the implementation of appendLine() with extra '\n'.

Fixes #3359

Change-Id: I7e88e74bd50f810cc27a496b2bb66999b7b0ed5d
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/691635
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Hongxiang Jiang <[email protected]>
Commit-Queue: Hongxiang Jiang <[email protected]>
kokoro-CI: kokoro <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>

Command which uses third party tool "gotests" will be marked as
"legacy" in both editor context menu and command palette. Legacy
command is "go.test.generate.function.legacy".

Instead, the original command will be pointing to the gopls's
implementation.

For #1594

Change-Id: If2d7ff67ff479ee82da3fd7322a6dad313306928
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/692575
Reviewed-by: Madeline Kalil <[email protected]>
kokoro-CI: kokoro <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

…ommand

The flakiness is caused by dlv substitute-path-guess-helper failed to
return within 2 seconds.

Using beforeEach and afterEach to create sandboxes and cleanup.

Instead of executing the dlv command, mock the guess function to make it
reliable.

Change-Id: Ib73aadc2d04d3d2f0c692def1565715869c6e9cf
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/694096
Reviewed-by: Robert Findley <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

Refactor logic to determine the latest version to make it more
clear.

Replace type assersion using <> with as.
https://google.github.io/styleguide/tsguide.html#type-and-non-nullability-assertions

For #2891

Change-Id: I2977fcf6e8f77b7a2b7f2aa409f2206685cf59bf
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/693676
Reviewed-by: Robert Findley <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Hongxiang Jiang <[email protected]>
Commit-Queue: Hongxiang Jiang <[email protected]>

Third party tools like gotests/gomodifytags can be triggered
from either commnad palette and context menu.

Gopls command can be triggered from command palette, context
menu and code action.

Differentiate between source trigger:
- Code Action: in language client middle ware, resolve code
action will resolve the code action to a command. Increase
the counter if the resolved command name is add test or modify
tags.
- Context Menu: vscode will pass uri (of type vscode.Uri) to
the command handler if it's triggered from context menu. The
uri is the uri to the file where the context menu is triggered.
- Command Palette: vscode will pass in undefined uri to the
command handler.

For golang/go#74869

Change-Id: I3a8b0cc46b916f72cbb59b7c2f764334d5d93e41
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/693015
Reviewed-by: Robert Findley <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Hongxiang Jiang <[email protected]>

vscode-go extension stores developer survey state (including the user
reaction and prompt time) in vscode memento. vscode-go will use the
previously stored state to determine whether we should prompt.

This CL does not change any developer survey prompting behavior, only
handles refactoring.

For #2891

Change-Id: Icac7d288c41b44a5492a33703956d8504e811b53
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/695395
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Robert Findley <[email protected]>

Fixes #3837

Change-Id: I46bbd7d3c4dea866bff2419a631f2407d6989835
GitHub-Last-Rev: 2d7676b
GitHub-Pull-Request: #3838
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/696355
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>
Reviewed-by: Carlos Amedee <[email protected]>

…dule

The fetched configuration will be stored in vscode momento with last
udpate time. The vscode-go extension will re-fetch if the cached config
is one day ago.

Local experiment at #2891 (comment)

For #2891

Change-Id: I8e6d2bd64ca8ccbc1f7ab1a2c0e87d5897200dea
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/697195
Reviewed-by: Robert Findley <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>

This patch fixes problem when user try debug not golang program, but see
"Go: Toggle Hide System Goroutines" command for threads in callstack view.

Change-Id: I3eb622fa6f6b913d107c50145d2061a0a977e7a5
GitHub-Last-Rev: 5d2a74e
GitHub-Pull-Request: #3778
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/679795
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Madeline Kalil <[email protected]>
Reviewed-by: Hongxiang Jiang <[email protected]>

For #2891

Change-Id: I54d017eaa53c94606171993560263c8c52d6afdd
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/697455
Auto-Submit: Hongxiang Jiang <[email protected]>
Reviewed-by: Todd Kulesza <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Reviewed-by: Robert Findley <[email protected]>

The config.json is now pointing at this year's survey waiting to
be tagged with survey/v0.1.0.

For #2891

Change-Id: Ia30e5f56a30085c213347b365b5a823491977f51
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/697395
Reviewed-by: Madeline Kalil <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>