Skip to content

Simplify upgrade code#1830

Merged
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:simplify-upgrades
Mar 19, 2019
Merged

Simplify upgrade code#1830
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:simplify-upgrades

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Mar 19, 2019

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

@codecov-io
Copy link
Copy Markdown

codecov-io commented Mar 19, 2019

Codecov Report

Merging #1830 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1830      +/-   ##
==========================================
- Coverage   45.42%   45.34%   -0.09%     
==========================================
  Files         143      143              
  Lines        6683     6570     -113     
==========================================
- Hits         3036     2979      -57     
+ Misses       3341     3313      -28     
+ Partials      306      278      -28
Impacted Files Coverage Δ
pkg/skaffold/schema/v1beta6/upgrade.go 56.75% <ø> (ø) ⬆️
pkg/skaffold/schema/v1beta5/upgrade.go 100% <100%> (+36.36%) ⬆️
pkg/skaffold/schema/v1beta4/upgrade.go 100% <100%> (+41.37%) ⬆️
pkg/skaffold/schema/v1beta1/upgrade.go 100% <100%> (+41.37%) ⬆️
pkg/skaffold/schema/v1beta3/upgrade.go 100% <100%> (+41.37%) ⬆️
pkg/skaffold/schema/v1beta2/upgrade.go 100% <100%> (+41.37%) ⬆️

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 f6c7f56...e9b0b5b. Read the comment docs.

Signed-off-by: David Gageot <david@gageot.net>
@dgageot dgageot force-pushed the simplify-upgrades branch from 7cd0e5c to e9b0b5b Compare March 19, 2019 12:31
// localdir/initImage
// helm useHelmSecrets
// 2. No removals
// 3. No updates
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: # 3. Updates://

@dgageot dgageot merged commit eb6a87d into GoogleContainerTools:master Mar 19, 2019
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.

4 participants