-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Update CONTRIBUTING.md. #1667
Update CONTRIBUTING.md. #1667
Conversation
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.
LGTM
.github/CONTRIBUTING.md
Outdated
@@ -64,7 +64,7 @@ $ ( cd integration && ../script/glide.sh get github.com/baz/quuz ) | |||
# generate (Only required to integrate other components such as web dashboard) | |||
$ go generate | |||
# Standard go build | |||
$ go build | |||
$ go build ./cmd/traefik | |||
# Using gox to build multiple platform | |||
$ gox "linux darwin" "386 amd64 arm" \ |
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'm wondering if we should kick this one out. Personally, I've never used it, but maybe it's just me...
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.
yeah, I agree too. We don't use gox anymore.
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.
Alright, removed.
d458ede
to
50d116c
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.
Thanks @timoreimann
LGTM
- Go 1.8 is the current minimum requirement. - The main binary moved to cmd/traefik. - Remove obsolete gox example.
50d116c
to
d742ddd
Compare
cmd/traefik
.