Skip to content

[Custom] [Deprecation] Use $IMAGE instead of $IMAGES #3084

Merged
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:fix-3081
Oct 21, 2019
Merged

[Custom] [Deprecation] Use $IMAGE instead of $IMAGES #3084
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:fix-3081

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Oct 18, 2019

Fixes #3081

$IMAGES is still passed, for compatibility reasons, but it no documented anymore

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

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 18, 2019

Codecov Report

Merging #3084 into master will not change coverage.
The diff coverage is 100%.

Impacted Files Coverage Δ
pkg/skaffold/build/custom/custom.go 62.5% <100%> (ø) ⬆️
pkg/skaffold/server/server.go 58.57% <0%> (ø) ⬆️

tejal29
tejal29 previously approved these changes Oct 18, 2019
@tejal29 tejal29 changed the title [Custom] Use $IMAGE instead of $IMAGES [Custom] [Deprecation] Use $IMAGE instead of $IMAGES Oct 18, 2019
@tejal29 tejal29 dismissed their stale review October 18, 2019 22:57

Please mention this in current deprecation notices.

@tejal29
Copy link
Copy Markdown
Contributor

tejal29 commented Oct 18, 2019

Can you please add these to current Deprecation notices to the Deprecation policy as well ?
https://skaffold-latest.firebaseapp.com/docs/references/deprecation/#current-deprecation-notices

We shd make sure this is captured in Release notes. Added Deprecation to the title.

@dgageot
Copy link
Copy Markdown
Contributor Author

dgageot commented Oct 21, 2019

Thanks @tejal29! I added a section in the Deprecation notices

Copy link
Copy Markdown
Contributor

@nkubala nkubala left a comment

Choose a reason for hiding this comment

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

two nits but LGTM

Fixes GoogleContainerTools#3081

Signed-off-by: David Gageot <david@gageot.net>
@dgageot dgageot merged commit 069bb55 into GoogleContainerTools:master Oct 21, 2019
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.

[Custom builder] $IMAGES is always a single image

4 participants