Skip to content

Update mailmap#2665

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
dmcgowan:update-mailmap
Sep 19, 2018
Merged

Update mailmap#2665
dmcgowan merged 1 commit intocontainerd:masterfrom
dmcgowan:update-mailmap

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented Sep 19, 2018

Add consistent contributor names and emails
before cleanup https://gist.github.com/dmcgowan/20bc5d9e19fd91136135586465e31e81

ping @nashasha1 @lifubang @jianliao82 @yanxuean @mirake please let me know if this is not how you want your name displayed in the release notes (name on the left is what gets used).

Add consistent contributor names and emails

Signed-off-by: Derek McGowan <[email protected]>
@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2665 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2665      +/-   ##
==========================================
- Coverage   44.94%   44.89%   -0.05%     
==========================================
  Files          92       92              
  Lines       10048    10232     +184     
==========================================
+ Hits         4516     4594      +78     
- Misses       4816     4915      +99     
- Partials      716      723       +7
Flag Coverage Δ
#linux 48.39% <ø> (-0.33%) ⬇️
#windows 41.67% <ø> (ø) ⬆️
Impacted Files Coverage Δ
oci/spec.go 93.75% <0%> (+0.06%) ⬆️
oci/spec_opts.go 21.46% <0%> (+0.18%) ⬆️
archive/compression/compression.go 58.51% <0%> (+15.87%) ⬆️

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 1ac5ac6...4d0e744. Read the comment docs.

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; can wait to merge until tomorrow for any responses from contributors on name format

@dmcgowan
Copy link
Copy Markdown
Member Author

Merging this for the rc, please comment here if anything doesn't look right and I will update for the final release notes.

@dmcgowan dmcgowan merged commit 308d3cb into containerd:master Sep 19, 2018
@yanxuean
Copy link
Copy Markdown
Member

@dmcgowan Please change to "Xuean Yan", thanks.

@lifubang
Copy link
Copy Markdown
Contributor

Yes, I have nothing to change.
my name and email is:
Lifubang [email protected]
Thanks.

@mirake
Copy link
Copy Markdown
Contributor

mirake commented Sep 20, 2018

@dmcgowan It looks great. Thanks.

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.

7 participants