Skip to content

MAINTAINERS: add Brandon Mitchell (sudo-bmitch) - #118

Merged
thaJeztah merged 1 commit into
opencontainers:masterfrom
AkihiroSuda:maintainers
Aug 6, 2026
Merged

MAINTAINERS: add Brandon Mitchell (sudo-bmitch)#118
thaJeztah merged 1 commit into
opencontainers:masterfrom
AkihiroSuda:maintainers

Conversation

@AkihiroSuda

Copy link
Copy Markdown
Member

@sudo-bmitch has been helping reviewing the PRs, so I'd like to invite him

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@AkihiroSuda
AkihiroSuda requested a review from a team July 21, 2026 14:41
@thaJeztah

Copy link
Copy Markdown
Member

Sigh.. it says there's an error without mentioning what the error is;

Screenshot 2026-07-21 at 16 48 13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds Brandon Mitchell (@sudo-bmitch) to the project’s maintainer and review ownership lists, aligning repository governance with his ongoing review contributions referenced in the PR description.

Changes:

  • Added Brandon Mitchell to MAINTAINERS.
  • Added @sudo-bmitch to the repository-wide CODEOWNERS entry.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
MAINTAINERS Adds Brandon Mitchell to the list of project maintainers.
CODEOWNERS Adds @sudo-bmitch as a code owner for all paths (*).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sudo-bmitch

sudo-bmitch commented Jul 21, 2026

Copy link
Copy Markdown

Sigh.. it says there's an error without mentioning what the error is;

Shows valid for me. Maybe a server glitch or it might be complaining that the maintainer list doesn't match the code owners.

image

@AkihiroSuda

Copy link
Copy Markdown
Member Author

https://github.com/AkihiroSuda/go-digest/blob/083ff5181652f78b7449dfe03dddcc61b6d1ae99/CODEOWNERS

image

@thaJeztah thaJeztah left a comment

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.

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

We can add you as well, @tianon 😅 🙈

Screenshot 2026-07-21 at 17 12 50

@tianon

tianon commented Jul 21, 2026

Copy link
Copy Markdown
Member

Nope, happy to see Brandon here, happy to see him welcomed 😂👍

@AkihiroSuda

Copy link
Copy Markdown
Member Author

ping @opencontainers/go-digest-maintainers

@thaJeztah

Copy link
Copy Markdown
Member

With @AkihiroSuda self included, we have 3/5 maintainers, missing @vbatts and @stevvooe - is that enough to go ahead?

@sudo-bmitch

Copy link
Copy Markdown

With @AkihiroSuda self included, we have 3/5 maintainers, missing @vbatts and @stevvooe - is that enough to go ahead?

That's a really good question. In the OCI specs, we require a 2/3 quorum. However that's described by the governance doc which is included in each spec from the template, and this project doesn't include that since it predates it: https://github.com/opencontainers/project-template/blob/main/GOVERNANCE.md

There's also a dangling reference in there to a motion to release that would allow 3 approvals with no rejects that can be used in projects that are not specs (which covers go-digest).

It may be easier to go the housekeeping route and reach out to @vbatts directly to see if he still wants to be a maintainer here, since he stepped back from other parts of OCI.

@thaJeztah

Copy link
Copy Markdown
Member

Yeah, I pretty much doubt there would be objections, and enough trust among folks involved here to know we don't do bad things, but also don't want to just skip the rules 😅

@AkihiroSuda

Copy link
Copy Markdown
Member Author

@vbatts @stevvooe PTAL 🙏

@thaJeztah

Copy link
Copy Markdown
Member

I didn't see him active on Slack, so tried to send Vincent a message on Mastodon (looks like he posted there recently)

@sudo-bmitch

Copy link
Copy Markdown

I emailed @vbatts and got a response, but no answer to whether he wants to remain a maintainer here or could give this a review. 🤷‍♂️

@vbatts vbatts left a comment

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.

LGTM
🧁!

@thaJeztah

Copy link
Copy Markdown
Member

Thanks Vincent! 🫶 Hope you're doing well!

@thaJeztah
thaJeztah merged commit e90d3c6 into opencontainers:master Aug 6, 2026
11 checks passed
@thaJeztah

Copy link
Copy Markdown
Member

@caniszczyk I don't think I have permissions to manage repo settings or to add maintainers; are you able to help out?

@caniszczyk

Copy link
Copy Markdown
Contributor

@thaJeztah invited!

@thaJeztah

Copy link
Copy Markdown
Member

Thank you! 🙌

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.

8 participants