Skip to content

Add readme to contib#1519

Merged
mlaventure merged 1 commit intocontainerd:masterfrom
crosbymichael:contrib
Sep 18, 2017
Merged

Add readme to contib#1519
mlaventure merged 1 commit intocontainerd:masterfrom
crosbymichael:contrib

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

Closes #1442

Signed-off-by: Michael Crosby [email protected]

@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 15, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1519   +/-   ##
=======================================
  Coverage   40.85%   40.85%           
=======================================
  Files          23       23           
  Lines        2964     2964           
=======================================
  Hits         1211     1211           
  Misses       1466     1466           
  Partials      287      287

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 e517952...58015ab. Read the comment docs.

Comment thread contrib/README.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe add their stability is not warranted or similar?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

they may or may not be, its still in the repo so we still have somewhat of a responsibility for the code

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐼

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM after typo fix

Comment thread contrib/README.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

minor typo: s/contib/contrib

End of sentence minor wording suggestion: "but still contribute to overall containerd usability." (drop the)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

Closes containerd#1442

Signed-off-by: Michael Crosby <[email protected]>
Copy link
Copy Markdown
Contributor

@mlaventure mlaventure left a comment

Choose a reason for hiding this comment

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

LGTM

@mlaventure mlaventure merged commit 0cb2c96 into containerd:master Sep 18, 2017
@crosbymichael crosbymichael deleted the contrib branch September 18, 2017 17:56
mauriciovasquezbernal pushed a commit to kinvolk/containerd that referenced this pull request Nov 13, 2020
config: TolerateMissingHugePagesCgroupController -> TolerateMissingHugetlbController
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