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

Replace Brendan Burns with Steve Lasker on maintainers. #844

Closed
wants to merge 1 commit into from

Conversation

brendandburns
Copy link

@brendandburns brendandburns commented Apr 21, 2021

This PR must get at least 5 LGTMs before merging:

Signed-off-by: Brendan Burns bburns@microsoft.com

Signed-off-by: Brendan Burns <bburns@microsoft.com>
@brendandburns
Copy link
Author

brendandburns commented Apr 21, 2021

LGTM (obviously)

Approved with PullApprove

cyphar
cyphar previously approved these changes Apr 22, 2021
@AkihiroSuda
Copy link
Member

needs rebase

@vbatts vbatts mentioned this pull request Jul 9, 2021
7 tasks
@vbatts
Copy link
Member

vbatts commented Jul 9, 2021

I'm not sure that these three PR's for maintainers accurately surface a maintainer election process.

It seems that while just barely, we have had a re-kindled interaction among maintainers recently, spurred by this initial audit.
Though two of the nominees (vrothberg and tych0) are not really present in reviews, PRs or the calls. Lasker is present in discussion and the calls, but not in contributions.

I move that we close these (#841 #842 #844) and among the active contributors as we continue rekindling, we have a proper nomination.

@vbatts vbatts mentioned this pull request Jul 9, 2021
7 tasks
@jonboulle
Copy link
Contributor

agree with vbatts - I think we can close these and reconsider "proper" nominations in the coming weeks if there's general sentiment it's desirable.

@dlorenc
Copy link

dlorenc commented Aug 14, 2021

Should this be closed?

This one is different from #842 since it's a resignation and a nomination. Is there any real reason they should be coupled?

@vbatts
Copy link
Member

vbatts commented Dec 2, 2021

Closing these three PRs (#841 #842 #844), so that adding/replacing maintainers can go through the regular nomination process

@vbatts vbatts closed this Dec 2, 2021
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.

7 participants