-
Notifications
You must be signed in to change notification settings - Fork 33
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
Fix Docker Action migration from v1 to v2 #42
Conversation
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
77190e0
to
662a8e2
Compare
Codecov Report
@@ Coverage Diff @@
## master #42 +/- ##
=======================================
Coverage 23.95% 23.95%
=======================================
Files 5 5
Lines 501 501
=======================================
Hits 120 120
Misses 359 359
Partials 22 22 Continue to review full report at Codecov.
|
I trusted too much the green check in CI and I didn't notice the warning: ``` Warning: Unexpected input(s) 'registry', 'tag_with_sha', valid inputs are ['builder', 'context', 'file', 'build-args', 'labels', 'tags', 'pull', 'target', 'allow', 'no-cache', 'platforms', 'load', 'push', 'outputs', 'cache-from', 'cache-to', 'secrets', 'github-token'] ``` This should fix the issue as explained here https://github.com/docker/build-push-action/blob/818fbc81014f50430ad3c990e14d744b09e3b80d/UPGRADE.md#tags-with-ref-and-git-labels Signed-off-by: Gianluca Arbezzano <[email protected]>
662a8e2
to
c185a3d
Compare
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
I trusted too much the green check in CI and I didn't notice the
warning:
This should fix the issue as explained here https://github.com/docker/build-push-action/blob/818fbc81014f50430ad3c990e14d744b09e3b80d/UPGRADE.md#tags-with-ref-and-git-labels