WIP: support Go language server (gopls)

.travis.yml

@@ -28,7 +28,6 @@ install:
   - go get -u golang.org/x/tools/cmd/goimports
   - go get -u golang.org/x/tools/cmd/gorename
   - go get -u github.com/sqs/goreturns
-  - go get -u github.com/mdempsky/gocode
   - go get -u github.com/alecthomas/gometalinter
   - go get -u github.com/zmb3/gogetdoc
   - go get -u github.com/zmb3/goaddimport

README.md

@@ -14,11 +14,9 @@
 This package includes the following functionality:
 
 - Display information about your current go installation, by running `go version` and `go env`
-- Autocomplete using `gocode`
-- Format your code with `gofmt`, `goimports`, or `goreturns`;
-  optionally run one of these tools on save of any `.go` file
-- Run `go install .` and `go test -c -o {tempdir} .` to verify your code compiles
-  and to keep `gocode` suggestions up to date
+- Autocompletion
+- Code formatting
+- Compile on save
 - Run a variety of linters (e.g. `golint`, `vet`, etc.) against your code using [`gometalinter`](https://github.com/alecthomas/gometalinter), [`revive`](https://github.com/mgechev/revive) or [`golangci-lint`](https://github.com/golangci/golangci-lint)
 - Run tests, display test output, and display test coverage using `go test -coverprofile`
 - Display documentation for identifiers in source code using
@@ -41,10 +39,6 @@ The following commands are run for the directory of the current file:
 - `go install .` (for normal `.go` files)
 - `go test -o {tmpdir} -c .` (for `_test.go` files)
 
-### Why Are You Running `go install` Instead Of `go build`?
-
-`gocode` (and a few other tools, like `gotype`) work on `.a` files (i.e. the package object archive), and the way to keep these up to date is to run `go install` periodically. This ensures your autocomplete suggestions are kept up to date.
-
 ## Platforms
 
 The package has CI for OS X, Windows and Ubuntu.
@@ -57,13 +51,12 @@ If you are missing any required tools, you may be prompted to install them. You
 go get -u golang.org/x/tools/cmd/goimports
 go get -u golang.org/x/tools/cmd/gorename
 go get -u github.com/sqs/goreturns
-go get -u github.com/mdempsky/gocode
+go get -u golang.org/x/tools/cmd/gopls
 go get -u github.com/alecthomas/gometalinter
 go get -u github.com/mgechev/revive
 go get -u github.com/golangci/golangci-lint/cmd/golangci-lint
 go get -u github.com/zmb3/gogetdoc
 go get -u github.com/zmb3/goaddimport
-go get -u github.com/rogpeppe/godef
 go get -u golang.org/x/tools/cmd/guru
 go get -u github.com/fatih/gomodifytags
 go get -u github.com/tpng/gopkgs

appveyor.yml

@@ -30,7 +30,6 @@ install:
   - go get -u golang.org/x/tools/cmd/goimports
   - go get -u golang.org/x/tools/cmd/gorename
   - go get -u github.com/sqs/goreturns
-  - go get -u github.com/mdempsky/gocode
   - go get -u github.com/alecthomas/gometalinter
   - go get -u github.com/zmb3/gogetdoc
   - go get -u github.com/zmb3/goaddimport

lib/build/builder.js

@@ -157,8 +157,8 @@ class Builder {
     }
     const srcDir = gopath + sep + 'src'
     return srcDir.split(sep).every((t, i) => cwd.split(sep)[i] === t)
-      ? 'install' // CWD is within gopath, `go install` to keep gocode up to date
-      : 'build' // CWD is outside gopath, `go build` will suffice
+      ? 'install' // CWD is within gopath, `go install`
+      : 'build' // CWD is outside gopath, `go build`
   }
 
   async lintInstall(cmd: string, options: ExecutorOptions) {
@@ -177,7 +177,6 @@ class Builder {
     }
 
     // Include the -i flag with go install.
-    // See: https://github.com/mdempsky/gocode/issues/79
     if (command === 'install' && !buildArgs.includes('-i')) {
       buildArgs.push('-i')
     }