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.

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

Add Jon Johnson as maintainer #288

Merged
merged 1 commit into from
Jul 9, 2021

Conversation

dmcgowan
Copy link
Member

@dmcgowan dmcgowan commented Jun 23, 2021

Jon Johnson has been helping out with the distribution spec work for some time and has deep knowledge of the protocol and history. As I stated during the audit, we should evaluate each candidate individually. Jon was proposed as a candidate before the audit so this vote is a bit overdue.

Requires 4 maintainer LGTM to reach 2/3 majority

stevvooe
stevvooe previously approved these changes Jun 23, 2021
Copy link
Contributor

@stevvooe stevvooe left a comment

Choose a reason for hiding this comment

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

LGTM

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

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@vbatts vbatts left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -1 +1 @@
* @dmcgowan @jzelinskie @jdolitsky @mikebrow @stevvooe @vbatts
* @dmcgowan @jzelinskie @jonjohnsonjr @jdolitsky @mikebrow @stevvooe @vbatts
Copy link
Member

Choose a reason for hiding this comment

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

unrelated question: could this line in the file just be * @opencontainers/distribution-spec-maintainers , so we don't have to manage this list in several places?

@vbatts vbatts merged commit d7ad9d5 into opencontainers:main Jul 9, 2021
@jdolitsky jdolitsky mentioned this pull request Sep 15, 2022
@sudo-bmitch sudo-bmitch mentioned this pull request Feb 1, 2024
8 tasks
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.

6 participants