Remove golint supported-tools (#4535)

golint has been deprecated and was archived in 2019

autoload/ale/linter.vim

@@ -42,7 +42,7 @@ let s:default_ale_linters = {
 \   'apkbuild': ['apkbuild_lint', 'secfixes_check'],
 \   'csh': ['shell'],
 \   'elixir': ['credo', 'dialyxir', 'dogma'],
-\   'go': ['gofmt', 'golint', 'gopls', 'govet'],
+\   'go': ['gofmt', 'gopls', 'govet'],
 \   'groovy': ['npm-groovy-lint'],
 \   'hack': ['hack'],
 \   'help': [],

doc/ale-go.txt

@@ -6,8 +6,8 @@ ALE Go Integration                                             *ale-go-options*
 Integration Information
 
 The `gometalinter` linter is disabled by default. ALE enables `gofmt`,
-`golint` and `go vet` by default. It also supports `staticcheck`, `go
-build`,  `gosimple`, `golangserver`.
+`gopls`, and `go vet` by default. It also supports `staticcheck, `go
+build`, `gosimple`, `golangserver`, and `golangci-lint`.
 
 To enable `gometalinter`, update |g:ale_linters| as appropriate:
 >
@@ -175,25 +175,6 @@ g:ale_go_golines_options                             *g:ale_go_golines_options*
   --max-length=100 (lines above 100 characters will be wrapped)
 
 
-===============================================================================
-golint                                                          *ale-go-golint*
-
-g:ale_go_golint_executable                         *g:ale_go_golint_executable*
-                                                   *b:ale_go_golint_executable*
-  Type: |String|
-  Default: `'golint'`
-
-  This variable can be set to change the golint executable path.
-
-
-g:ale_go_golint_options                               *g:ale_go_golint_options*
-                                                      *b:ale_go_golint_options*
-  Type: |String|
-  Default: `''`
-
-  This variable can be set to pass additional options to the golint linter.
-
-
 ===============================================================================
 gometalinter                                              *ale-go-gometalinter*
 

doc/ale-supported-languages-and-tools.txt

@@ -213,7 +213,6 @@ Notes:
   * `golangci-lint`!!
   * `golangserver`
   * `golines`
-  * `golint`
   * `gometalinter`!!
   * `gopls`
   * `gosimple`!!

doc/ale.txt

@@ -1641,7 +1641,7 @@ g:ale_linters                                                   *g:ale_linters*
   \   'apkbuild': ['apkbuild_lint', 'secfixes_check'],
   \   'csh': ['shell'],
   \   'elixir': ['credo', 'dialyxir', 'dogma'],
-  \   'go': ['gofmt', 'golint', 'gopls', 'govet'],
+  \   'go': ['gofmt', 'gopls', 'govet'],
   \   'groovy': ['npm-groovy-lint'],
   \   'hack': ['hack'],
   \   'help': [],
@@ -2987,7 +2987,6 @@ documented in additional help files.
     golangci-lint.........................|ale-go-golangci-lint|
     golangserver..........................|ale-go-golangserver|
     golines...............................|ale-go-golines|
-    golint................................|ale-go-golint|
     gometalinter..........................|ale-go-gometalinter|
     gopls.................................|ale-go-gopls|
     govet.................................|ale-go-govet|

supported-tools.md

@@ -222,7 +222,6 @@ formatting.
   * [golangci-lint](https://github.com/golangci/golangci-lint) :warning: :floppy_disk:
   * [golangserver](https://github.com/sourcegraph/go-langserver) :warning:
   * [golines](https://github.com/segmentio/golines)
-  * [golint](https://godoc.org/github.com/golang/lint)
   * [gometalinter](https://github.com/alecthomas/gometalinter) :warning: :floppy_disk:
   * [gopls](https://github.com/golang/go/wiki/gopls)
   * [gosimple](https://github.com/dominikh/go-tools/tree/master/cmd/gosimple) :warning: :floppy_disk:

test/handler/test_golangci_lint_handler.vader

@@ -16,17 +16,9 @@ Execute (The golangci-lint handler should handle names with spaces):
   \     'expected ''package'', found ''IDENT'' gibberish',
   \     'staticcheck',
   \   ],
-  \   [
-  \     'C:\something\file with spaces.go',
-  \     '37',
-  \     '5',
-  \     'expected ''package'', found ''IDENT'' gibberish',
-  \     'golint',
-  \   ],
   \ ],
   \ map(ale_linters#go#golangci_lint#GetMatches([
   \   'C:\something\file with spaces.go:12:3: expected ''package'', found ''IDENT'' gibberish (staticcheck)',
-  \   'C:\something\file with spaces.go:37:5: expected ''package'', found ''IDENT'' gibberish (golint)',
   \ ]), 'v:val[1:5]')
 
 Execute (The golangci-lint handler should handle paths correctly):
@@ -43,17 +35,9 @@ Execute (The golangci-lint handler should handle paths correctly):
   \     'type': 'W',
   \     'filename': ale#path#Simplify(expand('%:p:h') . '/test.go'),
   \   },
-  \   {
-  \     'lnum': 37,
-  \     'col': 5,
-  \     'text': 'expected ''package'', found ''IDENT'' gibberish (golint)',
-  \     'type': 'W',
-  \     'filename': ale#path#Simplify(expand('%:p:h') . '/test.go'),
-  \   },
   \ ],
   \ ale_linters#go#golangci_lint#Handler(bufnr(''), [
   \   file . ':12:3: expected ''package'', found ''IDENT'' gibberish (staticcheck)',
-  \   file . ':37:5: expected ''package'', found ''IDENT'' gibberish (golint)',
   \ ])
 
 Execute (The golangci-lint handler should handle only typecheck lines as errors):

test/test_filetype_linter_defaults.vader

@@ -36,7 +36,7 @@ Execute(The defaults for the elixir filetype should be correct):
   AssertEqual [], GetLinterNames('elixir')
 
 Execute(The defaults for the go filetype should be correct):
-  AssertEqual ['gofmt', 'golint', 'gopls', 'govet'], GetLinterNames('go')
+  AssertEqual ['gofmt', 'gopls', 'govet'], GetLinterNames('go')
 
   let g:ale_linters_explicit = 1