Skip to content

Simplify error messages#3997

Merged
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:shorter-err
Apr 21, 2020
Merged

Simplify error messages#3997
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:shorter-err

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Apr 21, 2020

Only keep what gives context and actionnable messages.

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

Only keep what gives context and actionnable
messages.

Signed-off-by: David Gageot <david@gageot.net>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 21, 2020

Codecov Report

Merging #3997 into master will not change coverage.
The diff coverage is 75.00%.

Impacted Files Coverage Δ
pkg/skaffold/docker/remote.go 22.22% <0.00%> (ø)
pkg/skaffold/build/local/local.go 60.00% <50.00%> (ø)
pkg/skaffold/docker/image.go 73.77% <50.00%> (ø)
pkg/skaffold/build/cache/lookup.go 100.00% <100.00%> (ø)
pkg/skaffold/build/parallel.go 93.93% <100.00%> (ø)
pkg/skaffold/build/sequence.go 100.00% <100.00%> (ø)
pkg/skaffold/docker/parse.go 90.37% <100.00%> (ø)
pkg/skaffold/runner/build_deploy.go 58.10% <100.00%> (ø)

@tejal29 tejal29 merged commit a12d62b into GoogleContainerTools:master Apr 21, 2020
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