Skip to content

Fix default-repo handling for images with non-alphabetic characters#1697

Merged
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
nkubala:default-repo-fix
Feb 28, 2019
Merged

Fix default-repo handling for images with non-alphabetic characters#1697
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
nkubala:default-repo-fix

Conversation

@nkubala
Copy link
Copy Markdown
Contributor

@nkubala nkubala commented Feb 26, 2019

Fixes #1657

This fixes an issue with the default-repo regex missing numeric characters, as well as hypens and underscores.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 26, 2019

Codecov Report

Merging #1697 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1697   +/-   ##
=======================================
  Coverage   46.55%   46.55%           
=======================================
  Files         125      125           
  Lines        5664     5664           
=======================================
  Hits         2637     2637           
  Misses       2754     2754           
  Partials      273      273
Impacted Files Coverage Δ
pkg/skaffold/util/image.go 100% <ø> (ø) ⬆️

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 b64a0ed...64583c9. Read the comment docs.

@dgageot dgageot merged commit 60173f6 into GoogleContainerTools:master Feb 28, 2019
@nkubala nkubala deleted the default-repo-fix branch June 12, 2019 21:41
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.

Default repo substitution doesn't work with the examples

5 participants