-
Notifications
You must be signed in to change notification settings - Fork 451
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
Refactor how images are pushed #138
Refactor how images are pushed #138
Conversation
2e39f13
to
5f4d970
Compare
Signed-off-by: Juraci Paixão Kröhling <[email protected]>
910dea7
to
f50eade
Compare
@kevinearls, would you review this one as well? |
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
|
||
- name: Docker meta | ||
id: docker_meta | ||
uses: crazy-max/ghaction-docker-meta@v1 |
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.
crazy-max? Should we be depending on someone named crazy-max? :-)
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 had the exact thought. Unfortunately, the official docs for the Docker action recommends this one dynamic labels and tags.
I'm merging, as there's only one way of knowing it works. |
…try#138) Signed-off-by: Juraci Paixão Kröhling <[email protected]>
…try#138) Signed-off-by: Juraci Paixão Kröhling <[email protected]>
Changed how images are pushed to registries, both for master and releases.