-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Error on invalid artifact workspace #4492
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4492 +/- ##
==========================================
- Coverage 72.37% 72.36% -0.01%
==========================================
Files 333 333
Lines 12944 12956 +12
==========================================
+ Hits 9368 9376 +8
- Misses 2980 2982 +2
- Partials 596 598 +2
Continue to review full report at Codecov.
|
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.
This LGTM!
…ploy (GoogleContainerTools#4657) * Revert GoogleContainerTools#4492 * Error in build when encountering an invalid artifact context * gofmt * Add tests * Hide util.Copy; fix up stray local testing bogons * revert to origin/master * gofmt
…ploy (GoogleContainerTools#4657) * Revert GoogleContainerTools#4492 * Error in build when encountering an invalid artifact context * gofmt * Add tests * Hide util.Copy; fix up stray local testing bogons * revert to origin/master * gofmt
Fixes: #4491
Description
Adds a validation step to ensure that all artifact workspaces exist.
Invalid location
File