-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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: move crosbymichael to EMERITUS; remove chief maintainer role #4324
Conversation
I talked to Michael, he says he is stepping down as a maintainer, being busy with other stuff. Thank you for all the hard work that you did! Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
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.
Respect
⭐ ⭐ ⭐ ⭐ ⭐
@@ -1,4 +1,3 @@ | |||
Michael Crosby <michael@thepasture.io> (@crosbymichael) |
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.
The chief maintainer has to be updated now
Lines 76 to 79 in 90ebcca
The Chief Maintainer for the project is responsible for overall architecture | |
of the project to maintain conceptual integrity. Large decisions and | |
architecture changes should be reviewed by the chief maintainer. | |
The current chief maintainer for the project is Michael Crosby (@crosbymichael). |
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.
Oh well. Maybe we can remove this position, since runc is mostly in the maintenance phase, with no major architectural decisions upcoming? WDYT @opencontainers/runc-maintainers ?
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.
👍 on removing the Chief Maintainer position, but I'm not sure whether "runc is mostly in the maintenance phase" is true.
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.
Yeah, the Chief Maintainer role is something we probably copied from the original maintainers file in Docker / Moby, it's probably fine to remove
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.
Added a commit removing the chief maintainer's position.
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.
Thanks for all of your hard work Michael!
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.
Thank you Michael!
Since Michael Crosby is stepping down, and we don't want to nominate someone else to be a chief maintainer, let's remove the position. Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
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
Thank you so much, @crosbymichael 🫶🫶
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 Michael for all of your hard work!
I talked to Michael, he says he is stepping down as a maintainer, being
busy with other stuff.
Thank you for all the hard work that you did!
Since Michael Crosby is stepping down, and we don't want to nominate
someone else to be a chief maintainer, let's remove the role.