-
Notifications
You must be signed in to change notification settings - Fork 6
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
Push to Docker Hub on merges to master? #19
Comments
If we also host e.g. ubuntu-vims on the Docker Hub repo, we could have automated builds for Vim master then (https://hub.docker.com/add/automated-build/testbed/). |
Hmm, it seems that I can't setup automated builds for the DH org. I can't figure out how to link a GH account to the DH org, and even if I did, it would have to be linked to my GH account which isn't ideal. A GH org might be needed as well. Am I missing something? |
Probably that would be the best. OTOH we basically only need a user and could do it from Travis then - without linking the services explicitly. Not sure if that's the best way though. Not really a pressing issue also, just something for the future. |
Ah, I get it now. Have Travis push to the DH repo after a successful run using encrypted env vars. I was getting mixed up between DH vs GH vs Travis service responsibilities. So, we would really just need a bot account on DH that would login from Travis. This is what I'm reading to draw this conclusion: https://sebest.github.io/post/using-travis-ci-to-build-docker-images/ |
Also, I don't know what org to use on GH that would sound canonical since |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
FWIW, https://hub.docker.com/r/testbed/vim is used by now, but still only manually. What would be required is setting up pushing from GitHub Actions, after #78. It could get triggered when a tag (or release) is created, and then either map / use the Git tag to / as |
I think we should make Travis push image updates automatically on merges, and not do it manually.
The text was updated successfully, but these errors were encountered: