Skip to content

Add Jon Johnson as maintainer#288

Merged
vbatts merged 1 commit intoopencontainers:mainfrom
dmcgowan:add-jon-johnson
Jul 9, 2021
Merged

Add Jon Johnson as maintainer#288
vbatts merged 1 commit intoopencontainers:mainfrom
dmcgowan:add-jon-johnson

Conversation

@dmcgowan
Copy link
Copy Markdown
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
Copy Markdown
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 <derek@mcg.dev>
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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