Skip to content

Add folders to tarballs#1878

Merged
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:tar-folders
Mar 27, 2019
Merged

Add folders to tarballs#1878
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:tar-folders

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Mar 26, 2019

If CreateTar is passed a folder, it shouldn't complain. The code currently complains on Jib projects built with GCB because Jib list folders as dependencies.

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

If `CreateTar` is passed a folder, it shouldn't
complain. The code currently complains on Jib
projects built with GCB because Jib list folders
as dependencies.

Signed-off-by: David Gageot <david@gageot.net>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1878 into master will decrease coverage by <.01%.
The diff coverage is 42.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1878      +/-   ##
==========================================
- Coverage    49.9%   49.89%   -0.01%     
==========================================
  Files         168      168              
  Lines        7354     7361       +7     
==========================================
+ Hits         3670     3673       +3     
- Misses       3328     3330       +2     
- Partials      356      358       +2
Impacted Files Coverage Δ
pkg/skaffold/util/tar.go 36.14% <42.85%> (+0.61%) ⬆️

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 d77c062...e2e72fe. Read the comment docs.

@dgageot dgageot merged commit 6012e3d into GoogleContainerTools:master Mar 27, 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