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

Remove warning for an image that’s built and used by fqn #713

Merged
merged 3 commits into from
Jun 25, 2018

Conversation

dgageot
Copy link
Contributor

@dgageot dgageot commented Jun 20, 2018

No description provided.

@dgageot dgageot force-pushed the fix-706 branch 3 times, most recently from 5679c22 to 23abe9a Compare June 20, 2018 16:22
@dgageot dgageot force-pushed the fix-706 branch 11 times, most recently from 2aab14f to 5300d08 Compare June 25, 2018 05:06
Copy link
Contributor

@dlorenc dlorenc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one request for a comment.

@@ -35,6 +35,8 @@ import (
"gopkg.in/yaml.v2"
)

var warner Warner = &logrusWarner{}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add a comment explaining that this is only used so that we can test warnings?

dgageot added 3 commits June 25, 2018 22:41
Image that is built and not used might be an error

Signed-off-by: David Gageot <[email protected]>
Signed-off-by: David Gageot <[email protected]>
@dgageot dgageot merged commit 31b9064 into GoogleContainerTools:master Jun 25, 2018
@dgageot dgageot deleted the fix-706 branch December 28, 2018 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants