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

Fix syntastic help #1323

Merged
merged 2 commits into from
Jun 7, 2017
Merged

Fix syntastic help #1323

merged 2 commits into from
Jun 7, 2017

Conversation

178inaba
Copy link
Contributor

@178inaba 178inaba commented Jun 7, 2017

syntastic can not use errcheck as a checker.

Sample vimrc:

let g:syntastic_go_checkers = ['golint', 'govet', 'errcheck']

Run :SyntasticInfo

Syntastic version: 3.8.0-56 (Vim 800, Darwin)
Info for filetype: go
...
Available checkers: go gofmt golint gometalinter govet
Currently enabled checkers: golint govet

When using errcheck with syntastic, you need to use gometalinter and specify errcheck as an option.

@fatih
Copy link
Owner

fatih commented Jun 7, 2017

Thanks @178inaba. This looks good. However I think the following would be better, what do you think? :

let g:syntastic_go_checkers = ['golint', 'govet']
let g:syntastic_mode_map = { 'mode': 'active', 'passive_filetypes': ['go'] }

If you want to add errcheck you can use gometalinter as a wrapper

let g:syntastic_go_checkers = ['golint', 'govet', 'gometalinter']
let g:syntastic_go_gometalinter_args = ['--disable-all', '--enable=errcheck']
let g:syntastic_mode_map = { 'mode': 'active', 'passive_filetypes': ['go'] }

@178inaba
Copy link
Contributor Author

178inaba commented Jun 7, 2017

@fatih Sounds good! I will add it!

@178inaba
Copy link
Contributor Author

178inaba commented Jun 7, 2017

Done.

@fatih fatih merged commit d75e175 into fatih:master Jun 7, 2017
@fatih
Copy link
Owner

fatih commented Jun 7, 2017

Thanks @178inaba 👍

@178inaba 178inaba deleted the help_syntastic branch June 7, 2017 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants