Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Kazuyoshi Kato as a reviewer #62

Merged
merged 1 commit into from
Jan 20, 2021
Merged

Add Kazuyoshi Kato as a reviewer #62

merged 1 commit into from
Jan 20, 2021

Conversation

estesp
Copy link
Member

@estesp estesp commented Jan 12, 2021 •
edited by mlaventure
Loading

Kazuyoshi has been increasing activity within the containerd project, reviewing PRs
and making meaningful contributions to the project. I propose @kzys be recognized
as a reviewer.

5 maintainer LGTM required (1/3) + new reviewer

Signed-off-by: Phil Estes estesp@gmail.com

AkihiroSuda
AkihiroSuda approved these changes Jan 12, 2021
View reviewed changes
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Welcome

dmcgowan
dmcgowan approved these changes Jan 12, 2021
View reviewed changes
Copy link
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp
Add Kazuyoshi Kato as a reviewer
26e5ee1 
Signed-off-by: Phil Estes <estesp@gmail.com>
fuweid
fuweid approved these changes Jan 12, 2021
View reviewed changes
Copy link
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

kzys
kzys approved these changes Jan 12, 2021
View reviewed changes
Copy link
Member

@kzys kzys left a comment

Choose a reason for hiding this comment

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

LGTM of course. Thanks all!

mxpv
mxpv approved these changes Jan 12, 2021
View reviewed changes
Copy link
Member

@mxpv mxpv left a comment

Choose a reason for hiding this comment

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

LGTM

jterry75
jterry75 approved these changes Jan 12, 2021
View reviewed changes
Copy link
Contributor

@jterry75 jterry75 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the contributions

crosbymichael
crosbymichael approved these changes Jan 12, 2021
View reviewed changes
Copy link
Member

@crosbymichael crosbymichael left a comment

Choose a reason for hiding this comment

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

LGTM

@AkihiroSuda
Copy link
Member

@mlaventure @stevvooe @dchen1107 @Random-Liu @mikebrow @yujuhong PTAL

@dmcgowan dmcgowan merged commit b73a06e into containerd:master Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kzys kzys kzys approved these changes

@dmcgowan dmcgowan dmcgowan approved these changes

@crosbymichael crosbymichael crosbymichael approved these changes

@mxpv mxpv mxpv approved these changes

@jterry75 jterry75 jterry75 approved these changes

@AkihiroSuda AkihiroSuda AkihiroSuda approved these changes

@mlaventure mlaventure mlaventure approved these changes

@fuweid fuweid fuweid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@estesp @AkihiroSuda @kzys @dmcgowan @crosbymichael @mxpv @jterry75 @mlaventure @fuweid