Skip to content

Use heroku/color for our colors#3757

Merged
dgageot merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:revamp-colors
Feb 29, 2020
Merged

Use heroku/color for our colors#3757
dgageot merged 2 commits intoGoogleContainerTools:masterfrom
dgageot:revamp-colors

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Feb 28, 2020

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

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 28, 2020

Codecov Report

Merging #3757 into master will decrease coverage by 0.05%.
The diff coverage is 90.9%.

Impacted Files Coverage Δ
pkg/skaffold/build/parallel.go 93.93% <100%> (-0.43%) ⬇️
cmd/skaffold/app/cmd/cmd.go 68.46% <100%> (+1.03%) ⬆️
pkg/skaffold/color/formatter.go 90% <90%> (-10%) ⬇️
pkg/skaffold/util/term.go 50% <0%> (-50%) ⬇️
...affold/kubernetes/portforward/kubectl_forwarder.go 65.85% <0%> (-2.44%) ⬇️

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

@tejal29 tejal29 left a comment

Choose a reason for hiding this comment

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

Did you forget to run go mod vendor ? I dont see any files added for the new dependency

@tejal29 tejal29 self-assigned this Feb 28, 2020
@dgageot
Copy link
Copy Markdown
Contributor Author

dgageot commented Feb 28, 2020

@tejal29 It was already vendored for pack

@dgageot dgageot merged commit a9c7b3a into GoogleContainerTools:master Feb 29, 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