src/goInstallTools: remove dlv-dap from missing tools
We hoped to use missing analysis tools to prompt users to update
dlv-dap since we want users to be using master. This results in
the Missing Analysis Tools notification to be constantly displayed,
which is distracting. We should replace this with other update
logic to keep users getting updates.
Change-Id: I71a99dc0e47ce1447acec2cda223ab10e3f3100e
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/306990
Trust: Suzy Mueller <suzmue@golang.org>
Run-TryBot: Suzy Mueller <suzmue@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
diff --git a/src/goInstallTools.ts b/src/goInstallTools.ts
index ab09bbf..287489e 100644
--- a/src/goInstallTools.ts
+++ b/src/goInstallTools.ts
@@ -567,17 +567,6 @@
(tool) =>
new Promise<Tool>((resolve, reject) => {
const toolPath = getBinPath(tool.name);
- if (tool.name === 'dlv-dap') {
- // Check if user already has dlv-dap binary.
- // If so, it's likely the user may be interested in updating the tool,
- // so we should mark it as important and return as a missing tool.
- if (path.isAbsolute(toolPath)) {
- tool.isImportant = true;
- resolve(tool);
- return;
- }
- tool.isImportant = false;
- }
resolve(path.isAbsolute(toolPath) ? null : tool);
})
)
