Skip to content

chore: Switch from gometalinter to golangci-lint#547

Merged
hairyhenderson merged 2 commits intohairyhenderson:masterfrom
nexeck:golangci
May 3, 2019
Merged

chore: Switch from gometalinter to golangci-lint#547
hairyhenderson merged 2 commits intohairyhenderson:masterfrom
nexeck:golangci

Conversation

@nexeck
Copy link
Contributor

@nexeck nexeck commented May 2, 2019

Github Issue: #546

@nexeck nexeck requested a review from hairyhenderson as a code owner May 2, 2019 16:58
@nexeck
Copy link
Contributor Author

nexeck commented May 2, 2019

I do not know how circle ci works. But it seems that the provided golangci-lint version is not the newest.

@nexeck
Copy link
Contributor Author

nexeck commented May 2, 2019

Please see: hairyhenderson/dockerfiles#55 for a fix for the lint error.

@hairyhenderson
Copy link
Owner

Thanks @nexeck! I'm going to add some commits to this PR - golangci-lint is significantly faster than gometalinter was, and so I'm going to remove the slow-lint target.

Signed-off-by: Dave Henderson <dhenderson@gmail.com>
@hairyhenderson hairyhenderson merged commit aa5bcba into hairyhenderson:master May 3, 2019
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