-
Notifications
You must be signed in to change notification settings - Fork 309
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
style: reformat all go files #437
Conversation
d1cc022
to
70122bd
Compare
2c7d103
to
cee0b27
Compare
cee0b27
to
8eb7a46
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, thanks for taking this up @developStorm !
One thing, could we add a make lint
similar to here so it's easier to do this locally without hunting for the gofmt
command?
Thanks for reviewing @phillip-stephens! Implemented in 18d5069. I did only go fmt since linter is out of scope of this PR... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran gofmt
myself and confirmed the change-set is the same. LGTM
And activate go formatter workflow