Skip to content

Handle additional cleanups from prior PR#2936

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:post-2622-cleanups
Jan 17, 2019
Merged

Handle additional cleanups from prior PR#2936
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:post-2622-cleanups

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Jan 16, 2019

Update comment and streamline getting the default platform spec; final two unresolved comments from #2622

// cc: @fuweid

Signed-off-by: Phil Estes [email protected]

Update comment and streamline getting the default platform spec

Signed-off-by: Phil Estes <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2936   +/-   ##
=======================================
  Coverage   44.02%   44.02%           
=======================================
  Files         102      102           
  Lines       10870    10870           
=======================================
  Hits         4785     4785           
  Misses       5351     5351           
  Partials      734      734
Flag Coverage Δ
#linux 47.65% <ø> (ø) ⬆️
#windows 41.24% <ø> (ø) ⬆️

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 df97414...b444d7d. Read the comment docs.

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 3acf6f1 into containerd:master Jan 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants