-
Notifications
You must be signed in to change notification settings - Fork 60
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
MAINTAINERS: add 2, remove 5 #54
Conversation
@opencontainers/go-digest-maintainers |
LGTM, thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
We have a stalled project, as we have a number of maintainers have moved on. We thank you all for your time. Going forward, I'm nominating 2 contributors that have a vested interest: * @thaJeztah * @AkihiroSuda And updating 2 emails to their community ones. Signed-off-by: Vincent Batts <[email protected]>
ping @opencontainers/go-digest-maintainers |
I think we should go ahead and merge this (looks like @stevvooe 's approve got unset after push though). Getting a quorum here of existing maintainers doesn't make sense considering the original purpose of this PR and there is no explicit rule requiring a majority to approve. We could have the TOB vote instead, but I don't think there is any issue here and if there is it can still be brought up to the TOB later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@caniszczyk can you be the judge here? |
Let's get this merged as there seems to be no issue, the TOB can resolve any issues in that unlikely scenario. |
We have a stalled project, as we have a number of maintainers have moved
on. We thank you all for your time.
Going forward, I'm nominating 2 contributors that have a vested
interest:
Signed-off-by: Vincent Batts [email protected]