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

Replace gometalinter by golangci-lint #71

Merged
merged 1 commit into from
Jan 19, 2019
Merged

Conversation

bpineau
Copy link
Owner

@bpineau bpineau commented Jan 19, 2019

Depending on so many external binaries (of unspecified version)
installed dynamically makes gometalinter very brittle: each CI
run may come up with linters variations.

gometalinter itself is not very stable; newer gometalinter
suddenly abandoning megacheck support was the last straw that
breaks the camel's back.

Icing on the cake, golangci-lint is much faster.

Depending on so many external binaries (of unspecified version)
installed dynamically makes gometalinter very brittle: each CI
run may come up with linters variations.

gometalinter itself is not very stable; newer gometalinter
suddenly abandoning megacheck support was the last straw that
breaks the camel's back.

Icing on the cake, golangci-lint is much faster.
@bpineau bpineau force-pushed the adopt-golangci-lint branch from 9709722 to e96cbd9 Compare January 19, 2019 18:28
@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.651% when pulling e96cbd9 on adopt-golangci-lint into 988edd4 on master.

@bpineau bpineau merged commit e6353dd into master Jan 19, 2019
@bpineau bpineau deleted the adopt-golangci-lint branch January 19, 2019 18:33
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