Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Added configuration for alternate gopls path. #2660

Merged
merged 1 commit into from
Jul 26, 2019

Conversation

anurbol
Copy link
Contributor

@anurbol anurbol commented Jul 18, 2019

This is to allow configuring an alternate path for gopls as well. Everything else in the extension is ready for this configuration, but the configuration feature itself is missing. By "ready" I mean these lines:

const goplsBinaryPath = getBinPath('gopls');

const alternateToolPath: string = alternateTools[tool];

This is to allow configuring an alternate path for gopls as well. Everything else in the extension is ready for this configuration, but the configuration feature itself is missing. By "ready" I mean this lines:
https://github.com/microsoft/vscode-go/blob/4b516f9a872f3a3086d6d56f8e346d4b5a7e53ef/src/goInstallTools.ts#L559
https://github.com/microsoft/vscode-go/blob/4b516f9a872f3a3086d6d56f8e346d4b5a7e53ef/src/util.ts#L375
Copy link
Contributor

@ramya-rao-a ramya-rao-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this is your first PR contribution to this project @anurbol, Thanks & Welcome!

Am curious as to what alternative are you hoping to use for gopls?

@ramya-rao-a ramya-rao-a merged commit cdaf5e8 into microsoft:master Jul 26, 2019
@anurbol
Copy link
Contributor Author

anurbol commented Jul 27, 2019

@ramya-rao-a Hi! I am making a tooling software and I want to reuse gopls' symbols to avoid parsing source files again. To do that I want to use a proxy between gopls and vscode-go. That proxy is the alternative (and I'll have to make it).

@ramya-rao-a
Copy link
Contributor

Understood, good luck!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants