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

staticcheck: check unused, gosimple #10442

Closed
gyuho opened this issue Feb 1, 2019 · 6 comments
Closed

staticcheck: check unused, gosimple #10442

gyuho opened this issue Feb 1, 2019 · 6 comments
Assignees

Comments

@gyuho
Copy link
Contributor

gyuho commented Feb 1, 2019

unused and gosimple have been deprecated (folded into staticcheck).

We have to make sure that we configure staticcheck to not miss any fmt tests from unused and gosimple, since I removed them in #10441.

@gyuho
Copy link
Contributor Author

gyuho commented Feb 1, 2019

@hexfusion @spzala Can you look into these?

@spzala
Copy link
Member

spzala commented Feb 1, 2019

@gyuho thanks and sure, I will look into it. @hexfusion please let me know if you are already looking at it. I may need some help with this but will let you know. Thanks!

@hexfusion
Copy link
Contributor

hexfusion commented Feb 1, 2019 via email

@spzala
Copy link
Member

spzala commented Apr 8, 2019

@gyuho hi, for this issue, I should wait on #10441 to be merged or is it the other way that 10441 is waiting on this one? Thanks!

@spzala
Copy link
Member

spzala commented Apr 30, 2019

Related PR - #10631

@spzala
Copy link
Member

spzala commented Jun 13, 2019

@gyuho hello, I am closing this one as we have updated tests to meet the staticcheck requirements under #10631. Thanks!

@spzala spzala closed this as completed Jun 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants