commit 3124d50af4bac81661e5e9174312d83730ea8fe1
author Rebecca Stambler <rstambler@golang.org> Mon Jun 22 13:10:41 2020 -0400
committer Rebecca Stambler <rstambler@golang.org> Mon Jun 22 17:18:55 2020 +0000
tree 914e9c9faa37aba48509e4cc930189089c4d2cd5
parent f5eb6edbe1e870439b21a3f3f239288654caef70

goLanguageServer: remove redundant code action provider

Fixes golang/vscode-go#247

Change-Id: I7b4a6bca3aa0868e5f40f6718147b4c0464da6d0
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/239284
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>

src/goLanguageServer.ts

diff --git a/src/goLanguageServer.ts b/src/goLanguageServer.ts
index e2afaca..53830cb 100644
--- a/src/goLanguageServer.ts
+++ b/src/goLanguageServer.ts
@@ -21,6 +21,7 @@
 } from 'vscode-languageclient';
 import WebRequest = require('web-request');
 import { extensionId } from './const';
+import { GoCodeActionProvider } from './goCodeAction';
 import { GoDefinitionProvider } from './goDeclaration';
 import { toolExecutionEnvironment } from './goEnv';
 import { GoHoverProvider } from './goExtraInfo';
@@ -334,6 +335,7 @@
 	);
 	defaultLanguageProviders.push(vscode.languages.registerRenameProvider(GO_MODE, new GoRenameProvider()));
 	defaultLanguageProviders.push(vscode.workspace.onDidChangeTextDocument(parseLiveFile, null, ctx.subscriptions));
+	defaultLanguageProviders.push(vscode.languages.registerCodeActionsProvider(GO_MODE, new GoCodeActionProvider()));
 
 	for (const provider of defaultLanguageProviders) {
 		ctx.subscriptions.push(provider);

src/goMain.ts

diff --git a/src/goMain.ts b/src/goMain.ts
index ac4fc8e..bff871e 100644
--- a/src/goMain.ts
+++ b/src/goMain.ts
@@ -11,7 +11,6 @@
 import { browsePackages } from './goBrowsePackage';
 import { buildCode } from './goBuild';
 import { check, notifyIfGeneratedFile, removeTestStatus } from './goCheck';
-import { GoCodeActionProvider } from './goCodeAction';
 import {
 	applyCodeCoverage, applyCodeCoverageToAllEditors, initCoverageDecorators, removeCodeCoverageOnFileSave,
 	toggleCoverageCurrentPackage, trackCodeCoverageRemovalOnFileChange, updateCodeCoverageDecorators
@@ -176,7 +175,6 @@
 	const testCodeLensProvider = new GoRunTestCodeLensProvider();
 	const referencesCodeLensProvider = new GoReferencesCodeLensProvider();
 
-	ctx.subscriptions.push(vscode.languages.registerCodeActionsProvider(GO_MODE, new GoCodeActionProvider()));
 	ctx.subscriptions.push(vscode.languages.registerCodeLensProvider(GO_MODE, testCodeLensProvider));
 	ctx.subscriptions.push(vscode.languages.registerCodeLensProvider(GO_MODE, referencesCodeLensProvider));
 	ctx.subscriptions.push(vscode.debug.registerDebugConfigurationProvider('go', new GoDebugConfigurationProvider()));