Skip to content

Every type of artifact should be handled.#2996

Merged
dgageot merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:diagnose-every-type
Oct 8, 2019
Merged

Every type of artifact should be handled.#2996
dgageot merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:diagnose-every-type

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Oct 7, 2019

If that’s not the case, we want to see it fail during tests.

Signed-off-by: David Gageot david@gageot.net

Reviewer Notes

  • The code flow looks good.
  • Unit test added.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 7, 2019

Codecov Report

Merging #2996 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted Files Coverage Δ
pkg/skaffold/runner/diagnose.go 12.67% <0%> (-0.76%) ⬇️

@dgageot dgageot force-pushed the diagnose-every-type branch 2 times, most recently from e264bb7 to 9e945d2 Compare October 7, 2019 08:56
If that’s not the case, we want to see it fail
during tests.

Signed-off-by: David Gageot <david@gageot.net>
@dgageot dgageot force-pushed the diagnose-every-type branch from 9e945d2 to 4408a80 Compare October 7, 2019 09:25
Signed-off-by: David Gageot <david@gageot.net>
@dgageot dgageot merged commit 69991f8 into GoogleContainerTools:master Oct 8, 2019
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.

3 participants