[release] 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 <[email protected]>
Run-TryBot: Suzy Mueller <[email protected]>
TryBot-Result: kokoro <[email protected]>
Reviewed-by: Hyang-Ah Hana Kim <[email protected]>
(cherry picked from commit d9fca4d)
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/306970

Author: suzmue

src/goInstallTools.ts

@@ -568,17 +568,6 @@ function getMissingTools(goVersion: GoVersion): Promise<Tool[]> {
 			(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);
 				})
 		)