commit | a30116df7a0fec62c14ff2fc6dea55c11692a5ea | [log] [tgz] |
---|---|---|
author | Rob Findley <rfindley@google.com> | Thu Feb 04 19:35:05 2021 -0500 |
committer | Robert Findley <rfindley@google.com> | Tue Feb 09 22:07:08 2021 +0000 |
tree | bf219c152c2d5cbd4947d1635e14cec9d0050787 | |
parent | fd2f9f3bd1e313f0e26c0fb36e8be6a2c4d17f51 [diff] |
internal/lsp: eliminate funcs from commands, and refactor appliesFn and suggestedFixFn were blocking eliminating the source.Command dynamic configuration. Remove them, and along the way refactor command dispatch to align better with the new internal/lsp/command package. This involved refactoring the internal/lsp/command.go as follows: - create a new commandHandler type, which will eventually implement command.Interface. - create a commandDeps struct to hold command dependencies. - move command functionality into methods on commandHandler. Of these, there are likely to be at least a couple points of controvery: I decided to store the ctx on the commandHandler, because I preferred it to threading a context through command.Interface when it isn't needed. We should revisit this in a later CL. I opted for a sparse commandDeps struct, rather than either explicit resolution of dependencies where necessary, or something more abstract like a proper dependency resolution pattern. It saved enough boilerplate that I deemed it worthwhile, but didn't want to commit to something more sophisticated. Actually switching to the internal/lsp/command package will happen in a later CL. Change-Id: I71502fc68f51f1b296bc529ee2885f7547145e92 Reviewed-on: https://go-review.googlesource.com/c/tools/+/289970 Trust: Robert Findley <rfindley@google.com> Run-TryBot: Robert Findley <rfindley@google.com> gopls-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Go Bot <gobot@golang.org> Reviewed-by: Heschi Kreinick <heschi@google.com>
This subrepository holds the source for various packages and tools that support the Go programming language.
Some of the tools, godoc
and vet
for example, are included in binary Go distributions.
Others, including the Go guru
and the test coverage tool, can be fetched with go get
.
Packages include a type-checker for Go and an implementation of the Static Single Assignment form (SSA) representation for Go programs.
The easiest way to install is to run go get -u golang.org/x/tools/...
. You can also manually git clone the repository to $GOPATH/src/golang.org/x/tools
.
This repository uses prettier to format JS and CSS files.
The version of prettier
used is 1.18.2.
It is encouraged that all JS and CSS code be run through this before submitting a change. However, it is not a strict requirement enforced by CI.
This repository uses Gerrit for code changes. To learn how to submit changes to this repository, see https://golang.org/doc/contribute.html.
The main issue tracker for the tools repository is located at https://github.com/golang/go/issues. Prefix your issue with “x/tools/(your subdir):” in the subject line, so it is easy to find.