-
Notifications
You must be signed in to change notification settings - Fork 25
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 formatting code with gofmt
#37
Comments
Here's an example of how gofmt would change the existing code - bbrowning@076935a It's mainly consistent whitespace and import ordering changes. |
Good idea! We should probably hack the Makefile to invoke it - maybe on the test target? |
Most projects set up CI validation for this. |
We use the following for this in our makefile. Happy to make a PR
|
@maleck13 that'd be awesome if you could thanks! |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Many Golang project use
gofmt
to consistently format their Go code. And, it's not uncommon to setup your editor to automatically rungofmt
when saving changes to a file. Perhaps we should consider doing so as well?The text was updated successfully, but these errors were encountered: