-
Notifications
You must be signed in to change notification settings - Fork 226
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
nonspec: Remove Kris from MAINTAINERS #1174
Conversation
Signed-off-by: Tom Hennen <[email protected]>
✅ Deploy Preview for slsa canceled.
|
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.
👍
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
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.
I appreciated all of Kris's contributions to the community. I would definitely welcome future collaborations!
100% Thank you for making this explicit. I should have done so myself. @kpk47's efforts are very much appreciated. |
Since I proposed this I believe we've met the majority maintainer approval threshold (my sending the PR counts as approval IMO). I'll go ahead and merge. |
@kpk47 has been inactive for some time so I figured I'd send this to clean things up since we've had some new maintainers added recently.
However if Kris would still be interested in contributing we could drop this.
According to our governance this PR requires majority approval