Skip to content

Remove a TODO from the code comments that is complete#2669

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:fix-withuser-comment
Sep 21, 2018
Merged

Remove a TODO from the code comments that is complete#2669
crosbymichael merged 1 commit intocontainerd:masterfrom
estesp:fix-withuser-comment

Conversation

@estesp
Copy link
Copy Markdown
Member

@estesp estesp commented Sep 20, 2018

WithUser... helpers do support non-snapshot rootfs now.

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

WithUser... helpers do support non-snapshot rootfs now.

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

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2669   +/-   ##
=======================================
  Coverage   45.04%   45.04%           
=======================================
  Files          92       92           
  Lines       10106    10106           
=======================================
  Hits         4552     4552           
  Misses       4834     4834           
  Partials      720      720
Flag Coverage Δ
#linux 48.78% <ø> (+0.02%) ⬆️
#windows 41.8% <ø> (ø) ⬆️

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 308d3cb...41615e8. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@ehazlett
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 6496078 into containerd:master Sep 21, 2018
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