Skip to content

Skaffold modules design proposal#4350

Merged
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
balopat:skaffold-modules-proposal
Jul 7, 2020
Merged

Skaffold modules design proposal#4350
tejal29 merged 1 commit intoGoogleContainerTools:masterfrom
balopat:skaffold-modules-proposal

Conversation

@balopat
Copy link
Contributor

@balopat balopat commented Jun 18, 2020

Please comment on the doc for this design proposal!

@balopat balopat requested a review from a team as a code owner June 18, 2020 22:53
@balopat balopat requested a review from dgageot June 18, 2020 22:53
@codecov
Copy link

codecov bot commented Jun 18, 2020

Codecov Report

Merging #4350 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4350   +/-   ##
=======================================
  Coverage   71.74%   71.75%           
=======================================
  Files         324      324           
  Lines       12511    12515    +4     
=======================================
+ Hits         8976     8980    +4     
  Misses       2966     2966           
  Partials      569      569           
Impacted Files Coverage Δ
pkg/skaffold/debug/cnb.go 93.10% <0.00%> (+0.51%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4baad7c...3d55095. Read the comment docs.

Copy link
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.

This looks great! Looks like you have captured all of the user cases nice!
Thank you!

@tejal29
Copy link
Contributor

tejal29 commented Jun 22, 2020

@balopat please feel free to merge this in once you have resolved all the issues.

@tejal29 tejal29 merged commit 3d19c7c into GoogleContainerTools:master Jul 7, 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