-
Notifications
You must be signed in to change notification settings - Fork 897
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
Add Makefile and Travis CI build #200
Conversation
9df3fde
to
7a9f573
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.
Nice, and good to see that misspell
is smart enough not to give us false negatives for words like "distributedcontext".
.gitignore
Outdated
@@ -8,6 +8,10 @@ | |||
.settings | |||
bin | |||
|
|||
# Go module | |||
go.mod |
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.
for my curiosity, why this? We may need to update this.
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.
On a second thought I agree it's better to have the modules. Also added a tools.go
to follow the best practice on having a consistent version on tools in 7b79cea.
* Add Makefile and Travis CI build * Simplify the script * Add to contributing guideline * Include Go modules and tools.go * Remove new line at eof
* Add Makefile and Travis CI build * Simplify the script * Add to contributing guideline * Include Go modules and tools.go * Remove new line at eof
Follow up of #199 (comment).
Need repo admins to set up Travis after this commit.