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

fix: fix kubectl result formatting for debug logs #7293

Merged

Conversation

briandealwis
Copy link
Member

DEBU[0081] 2manifests to deploy.2are updated or new      subtask=0 task=Deploy

@codecov
Copy link

codecov bot commented Apr 12, 2022

Codecov Report

Merging #7293 (39e9bb4) into main (290280e) will decrease coverage by 1.98%.
The diff coverage is 56.96%.

@@            Coverage Diff             @@
##             main    #7293      +/-   ##
==========================================
- Coverage   70.48%   68.49%   -1.99%     
==========================================
  Files         515      560      +45     
  Lines       23150    26507    +3357     
==========================================
+ Hits        16317    18157    +1840     
- Misses       5776     7095    +1319     
- Partials     1057     1255     +198     
Impacted Files Coverage Δ
cmd/skaffold/app/cmd/deploy.go 52.00% <ø> (-1.85%) ⬇️
cmd/skaffold/app/cmd/dev.go 84.61% <0.00%> (ø)
cmd/skaffold/app/cmd/render.go 36.66% <0.00%> (-4.72%) ⬇️
cmd/skaffold/skaffold.go 0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/inspect_tests.go 62.50% <14.28%> (-1.14%) ⬇️
cmd/skaffold/app/cmd/lsp.go 28.12% <28.12%> (ø)
cmd/skaffold/app/cmd/fix.go 68.85% <40.00%> (-7.62%) ⬇️
cmd/skaffold/app/cmd/lint.go 42.85% <42.85%> (ø)
cmd/skaffold/app/cmd/find_configs.go 48.88% <50.00%> (+0.24%) ⬆️
cmd/skaffold/app/skaffold.go 76.19% <70.00%> (-8.43%) ⬇️
... and 225 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@MarlonGamez MarlonGamez merged commit 0ea61f8 into GoogleContainerTools:main Apr 12, 2022
bskaplan pushed a commit to bskaplan/skaffold that referenced this pull request Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants