Skip to content
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 allow users the ability to pull a container in Github actions #115

Closed
wants to merge 9 commits into from

Conversation

TheQueenIsDead
Copy link

In response to this StackOverflow question and #42, publishing a Docker image for users of this action will allow them to directly pull the image instead of having to build the source code each time. This reduces the time to prepare the action from ~30 seconds to ~8

To enable this change to work, you will have to create a new repository in DockerHub, and add your username and password to the Secrets configuration in GitHub

It's worth noting that you can publish an image to the GitHub repository if you wish, and the instructions for that can be found here:
https://docs.github.com/en/actions/guides/publishing-docker-images

To view an example of what this change enables, please see the changes made in this PR in conjunction with the DockerHub image, and an example use case:

Example Usage: https://github.com/TheQueenIsDead/action-caching/blob/main/.github/workflows/push.yml
Example Repo Workflow: https://github.com/TheQueenIsDead/action-caching/runs/3259818711?check_suite_focus=true
(Please be aware the example workflow failed, but after the fact, and seemingly due to the fact that my commit message includes characters that are perhaps not being handled. The Setup Job and Pull thequeenisdead/gh-action-bump-version steps are the most important things to look at)

@TheQueenIsDead TheQueenIsDead mentioned this pull request Aug 6, 2021
@TheQueenIsDead
Copy link
Author

@phips28 , please close this if you're going to be going down the route of an NPM action instead of a Dockerised one 👍

@phips28
Copy link
Owner

phips28 commented Sep 2, 2021

#118

@phips28 phips28 closed this Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants