Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

x/tools/gopls: centralize all "go env X" calls #36487

Closed
stamblerre opened this issue Jan 9, 2020 · 1 comment
Closed

x/tools/gopls: centralize all "go env X" calls #36487

stamblerre opened this issue Jan 9, 2020 · 1 comment
Labels
FrozenDueToAge gopls Issues related to the Go language server, gopls. NeedsFix The path to resolution is known, but the work has not been done. Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
gopls/v0.4.0

Comments

@stamblerre
Copy link
Contributor

stamblerre commented Jan 9, 2020 •
edited

We've started relying on a number of calls to go env to build up various pieces of the view. At minimum right now, we call go env GOPATH for the goimports process env function, go env GOMOD for the -modfile handling, and now go env GOCACHE to detect test mains. All of these should be consolidated into a single go env call that happens on view creation.
@gopherbot gopherbot added this to the Unreleased milestone Jan 9, 2020
@gopherbot gopherbot added Tools This label describes issues relating to any tools in the x/tools repository. gopls Issues related to the Go language server, gopls. labels Jan 9, 2020
@stamblerre stamblerre added the NeedsFix The path to resolution is known, but the work has not been done. label Jan 9, 2020
@stamblerre stamblerre modified the milestones: Unreleased, gopls v1.0 Jan 9, 2020
@gopherbot
Copy link

Change https://golang.org/cl/214417 mentions this issue: internal/lsp: push initialization tasks into one function

@stamblerre stamblerre modified the milestones: gopls/v1.0.0, gopls/v0.4.0 Jul 22, 2020
@golang golang locked and limited conversation to collaborators Jul 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge gopls Issues related to the Go language server, gopls. NeedsFix The path to resolution is known, but the work has not been done. Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Milestone
gopls/v0.4.0
Development

No branches or pull requests
2 participants
@stamblerre @gopherbot