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

[V3] add renderer basic struct. #5793

Merged
merged 1 commit into from
May 24, 2021

Conversation

yuwenma
Copy link
Contributor

@yuwenma yuwenma commented May 5, 2021

Fixes: #nnn
Related: #5673
Merge after: #5865

Description
The renderer struct initialize generator which parses the DRY config and stores the dry config in a tmp directory.

To reviewers:
this PR is diff-based from #5865, to review the renderer specific code change, please see this 4867900 (#5793)

@yuwenma yuwenma requested a review from a team as a code owner May 5, 2021 20:54
@yuwenma yuwenma requested a review from briandealwis May 5, 2021 20:54
@google-cla google-cla bot added the cla: yes label May 5, 2021
@yuwenma yuwenma requested review from tejal29 and removed request for briandealwis and tejal29 May 5, 2021 20:55
@yuwenma yuwenma marked this pull request as draft May 5, 2021 20:56
@codecov
Copy link

codecov bot commented May 5, 2021

Codecov Report

Merging #5793 (8a8f7a5) into master (4eb8930) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5793      +/-   ##
==========================================
- Coverage   70.90%   70.88%   -0.02%     
==========================================
  Files         447      448       +1     
  Lines       16908    16932      +24     
==========================================
+ Hits        11988    12002      +14     
- Misses       4029     4034       +5     
- Partials      891      896       +5     
Impacted Files Coverage Δ
pkg/skaffold/render/renderer/renderer.go 50.00% <50.00%> (ø)
pkg/skaffold/docker/parse.go 86.19% <0.00%> (-0.96%) ⬇️
pkg/skaffold/util/tar.go 56.00% <0.00%> (+5.33%) ⬆️

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 4eb8930...8a8f7a5. Read the comment docs.

@yuwenma yuwenma marked this pull request as ready for review May 19, 2021 22:45
@yuwenma yuwenma requested review from MarlonGamez and tejal29 May 19, 2021 22:52
@yuwenma yuwenma force-pushed the render-1-basic branch 2 times, most recently from 5cd8c68 to 41093fd Compare May 20, 2021 01:19
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.

Please rebase and a qq.

pkg/skaffold/render/renderer/renderer.go Show resolved Hide resolved
@tejal29 tejal29 requested review from MarlonGamez and removed request for MarlonGamez May 20, 2021 16:15
The renderer struct initialize generator which parses the DRY config and stores the dry config in a tmp directory.
@tejal29 tejal29 merged commit e030a85 into GoogleContainerTools:master May 24, 2021
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.

2 participants