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

Consider pinning https://github.com/golangci/golangci-lint/ #699

Closed
dlorenc opened this issue Jun 18, 2018 · 5 comments
Closed
Labels
kind/bug Something isn't working meta/testing Issues focused on testing Skaffold itself priority/p3 agreed that this would be good to have, but no one is available at the moment.

Comments

@dlorenc
Copy link
Contributor

dlorenc commented Jun 18, 2018

We just had an issue where golangci-lint started failing in CI because of some new validation rules they added. This is a bad experience for contributors who start to see failures in CI unrelated to their changes.

We should consider pinning to a release available at https://github.com/golangci/golangci-lint/releases

@dlorenc
Copy link
Contributor Author

dlorenc commented Jun 18, 2018

Ref #697

@dgageot
Copy link
Contributor

dgageot commented Jun 19, 2018

@dlorenc I'd love to do that. How can we pin a given version though?

@dgageot dgageot added kind/bug Something isn't working area/testing Issues concerning the testing phase of Skaffold labels Jun 19, 2018
@dlorenc
Copy link
Contributor Author

dlorenc commented Jun 19, 2018

For travis we can grab one of the releases from https://github.com/golangci/golangci-lint/releases with curl/tar and then validate the sha. Or we could go get then cd/git checkout.

@r2d4
Copy link
Contributor

r2d4 commented Jun 19, 2018

I did this for dep

https://github.com/GoogleContainerTools/skaffold/blob/master/hack/dep.sh#L19-L30

Although this method fails when you have an old version of dep already on your path. Theres probably some extra work to be done to verify the installed version is correct

@r2d4 r2d4 added the priority/p3 agreed that this would be good to have, but no one is available at the moment. label Jun 29, 2018
@jirfag
Copy link

jirfag commented Jun 30, 2018

Hi!
Here https://github.com/golangci/golangci-lint/blob/master/README.md#install are commands for pinning a release, it has installation script. Also it much faster than go get

@nkubala nkubala added meta/testing Issues focused on testing Skaffold itself and removed area/testing Issues concerning the testing phase of Skaffold labels Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working meta/testing Issues focused on testing Skaffold itself priority/p3 agreed that this would be good to have, but no one is available at the moment.
Projects
None yet
Development

No branches or pull requests

5 participants