commit | c5fb716d6688a859aae56d26d3e6070808df29f7 | [log] [tgz] |
---|---|---|
author | Dimitri Sokolyuk <sokolyuk@gmail.com> | Fri Jun 02 16:41:31 2017 -0700 |
committer | Joe Tsai <joetsai@digital-static.net> | Fri Jun 02 16:41:31 2017 -0700 |
tree | 6fe1c94a458ecf658708107bff1d400a765d3f52 | |
parent | a5f4a247366d8fc436941822e14fc3eac7727015 [diff] |
Fix lint on packages with all generated files A package with all generated files will reduce to a set with no source files, which causes lint to panic on a nil-pointer dereference. Move the conditional check for an empty package until after we have checked the package for generated files. Fixes #300
Golint is a linter for Go source code.
Golint requires Go 1.6 or later.
go get -u github.com/golang/lint/golint
Invoke golint
with one or more filenames, directories, or packages named by its import path. Golint uses the same import path syntax as the go
command and therefore also supports relative import paths like ./...
. Additionally the ...
wildcard can be used as suffix on relative and absolute file paths to recurse into them.
The output of this tool is a list of suggestions in Vim quickfix format, which is accepted by lots of different editors.
Golint differs from gofmt. Gofmt reformats Go source code, whereas golint prints out style mistakes.
Golint differs from govet. Govet is concerned with correctness, whereas golint is concerned with coding style. Golint is in use at Google, and it seeks to match the accepted style of the open source Go project.
The suggestions made by golint are exactly that: suggestions. Golint is not perfect, and has both false positives and false negatives. Do not treat its output as a gold standard. We will not be adding pragmas or other knobs to suppress specific warnings, so do not expect or require code to be completely “lint-free”. In short, this tool is not, and will never be, trustworthy enough for its suggestions to be enforced automatically, for example as part of a build process. Golint makes suggestions for many of the mechanically checkable items listed in Effective Go and the CodeReviewComments wiki page.
If you find an established style that is frequently violated, and which you think golint could statically check, file an issue.
Contributions to this project are welcome, though please send mail before starting work on anything major. Contributors retain their copyright, so we need you to fill out a short form before we can accept your contribution.
Add this to your ~/.vimrc:
set rtp+=$GOPATH/src/github.com/golang/lint/misc/vim
If you have multiple entries in your GOPATH, replace $GOPATH
with the right value.
Running :Lint
will run golint on the current file and populate the quickfix list.
Optionally, add this to your ~/.vimrc
to automatically run golint
on :w
autocmd BufWritePost,FileWritePost *.go execute 'Lint' | cwindow
Add this to your .emacs
file:
(add-to-list 'load-path (concat (getenv "GOPATH") "/src/github.com/golang/lint/misc/emacs")) (require 'golint)
If you have multiple entries in your GOPATH, replace $GOPATH
with the right value.
Running M-x golint will run golint on the current file.
For more usage, see Compilation-Mode.