Remove inactive maintainers #108
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Christian Hernandez <christian@codefresh.io>
4 tasks
Member
Author
|
+1 Binding |
scottrigby
requested changes
Jul 1, 2022
Member
scottrigby
left a comment
scottrigby
left a comment
There was a problem hiding this comment.
Its true project governance does say that – among other maintainer responsibilities – maintainers must:
- Maintain involvement and open collaboration with working group members and committees
- Ask for help when unsure and step down considerately when appropriate
But a few things before merging this or any PR like it:
- personally, I would ideally like to hear from @chrispat and @csand-msft on this PR about their upcoming availability before removing either of them from the maintainers list. If we can not reach any maintainer for a period of time (perhaps a few months?) then I agree at some point it would make sense to remove without hearing back. But I don't think we're there yet
- It would make sense to separate these PRs individually, if we hear back from one person and not the other. Because they are not really tied together for any specific reason
- When/if a maintainer is removed, let's add them to an alumni section somewhere. We currently have a GitHub team called
alumniwith previous maintainers on it. I think it would be nice to add that to this or another file just to publicly recognize their service
Member
|
CC #115 I'm unclear on the process to remove maintainers per the governance doc today. |
Member
|
This will be resolved in a separate PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR removes inactive maintainers from the maintainers' file.
Context: #107
Signed-off-by: Christian Hernandez christian@codefresh.io