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

make it part of golang ci lint #2

Closed
kishaningithub opened this issue Dec 1, 2019 · 2 comments
Closed

make it part of golang ci lint #2

kishaningithub opened this issue Dec 1, 2019 · 2 comments

Comments

@kishaningithub
Copy link

@kishaningithub kishaningithub commented Dec 1, 2019

IMHO this great tool will reach a far wider audience if it is made part of https://github.com/golangci/golangci-lint

@fatih

This comment has been minimized.

Copy link
Owner

@fatih fatih commented Dec 1, 2019

Thank for your feedback. Unfortunately I don't have the bandwith or time to work on this. This is a side project I've done in my spare times. Feel free to use it this way. Thank you.

@fatih fatih closed this Dec 1, 2019
@ferhatelmas

This comment has been minimized.

Copy link

@ferhatelmas ferhatelmas commented Dec 3, 2019

@fatih I can try to implement the integration. It might help with larger adaptation.

However, errwrap.Analyzer must be put outside of the internal package so that it could be used by the golangci-lint. Good for a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.