-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
Require gofmt on Travis build #761
Conversation
fe4068b
to
5cd3ab0
Compare
cd80fe7
to
6bb97c9
Compare
6bb97c9
to
e4657c9
Compare
This reverts commit e4657c9.
#!/bin/bash | ||
|
||
cd "$(dirname $0)" | ||
if [ -n "$(go fmt ./...)" ]; then |
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.
I yanked google/gopacket's approach.
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.
Lovely!
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.
This is sweet, thank you! 💙
#!/bin/bash | ||
|
||
cd "$(dirname $0)" | ||
if [ -n "$(go fmt ./...)" ]; then |
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.
Lovely!
Closes #760