-
Notifications
You must be signed in to change notification settings - Fork 114
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
Publish docker image to docker hub? #49
Comments
brandur-stripe
pushed a commit
that referenced
this issue
Mar 21, 2018
Pushes the built Docker image to Docker Hub as part of a Travis deploy step. This allows Docker users to access the image more easily. There are a few caveats with this implementation: 1. We're only building `master`. If this ends up being something we use, we should build tagged version images as well. 2. It pushes to `stripemock/stripe-mock` instead of `stripe/stripe-mock` because we're having a little trouble getting the collaboration privileges worked out on Docker Hub's Stripe organization right now. We should change it to `stripe/stripe-mock` when possible. Fixes #49.
brandur-stripe
pushed a commit
that referenced
this issue
Mar 21, 2018
Pushes the built Docker image to Docker Hub as part of a Travis deploy step. This allows Docker users to access the image more easily. There are a few caveats with this implementation: 1. We're only building `master`. If this ends up being something we use, we should build tagged version images as well. 2. It pushes to `stripemock/stripe-mock` instead of `stripe/stripe-mock` because we're having a little trouble getting the collaboration privileges worked out on Docker Hub's Stripe organization right now. We should change it to `stripe/stripe-mock` when possible. Fixes #49.
I added #50 to push an image to Docker Hub based off the A couple caveats:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an official build of the docker image that can be used without building? I couldn't find one on docker hub.
The text was updated successfully, but these errors were encountered: