Skip to content

Updated GitHub Action#259

Open
danielwagn3r wants to merge 2 commits intoimapsync:masterfrom
danielwagn3r:master
Open

Updated GitHub Action#259
danielwagn3r wants to merge 2 commits intoimapsync:masterfrom
danielwagn3r:master

Conversation

@danielwagn3r
Copy link
Contributor

Enable conditional push to Docker Hub (only for release tags)
Enable Dependabot

Enable conditional push to Docker Hub (only for release tags)
Enable Dependabot
@danielwagn3r
Copy link
Contributor Author

As promised a reworked GitHub Action.

Required Secrets:

  • DOCKERHUB_IMAGENAME, e.g gilleslamiral/imapsync
  • DOCKERHUB_TOKEN, i.e a Docker Hub Personal Access Token to be able to push images
  • DOCKERHUB_USERNAME, e.g. gilleslamiral

The action runs on every push to master, but only pushes if a new matching tag was created - the tag ist then used for the Docker tag as well.

@danielwagn3r
Copy link
Contributor Author

If you somewhen plan have the latest imapsync version on GitHub the Dockerfile can be optimized further, to get rid of the wget which is currently used.

Fixed missing procps package which is a dependency.
@gilleslamiral
Copy link
Member

Thanks for this request.
The problem is that I don't want to trigger a Docker image creation and publishing without testing it before.

@danielwagn3r
Copy link
Contributor Author

Full understandable, I'm used to this as well. Therefore, the proposed Action only triggers when a release is tagged in the Git repository.

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