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
Clarify champion selection #905
base: main
Are you sure you want to change the base?
Conversation
+1, LGTM! Since the section is about "things that help" it could even be worth calling out that a core maintainer as champion is an even stronger signal than a maintainer alone and no one from core involved. |
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 @matthewp ! Typo, and a suggestion for laying things out a little more explicitly!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
|
||
**Location:** GitHub Issues [(see all accepted proposals).](https://github.com/withastro/roadmap/issues) | ||
|
||
**What to Expect:** A proposal reaches this stage (aka "is accepted") during a meeting with Maintainers and TSC, following our existing [RFC Proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) voting process. | ||
|
||
When a proposal is accepted, a TSC member will create a new GitHub Issue summarizing the original proposal using our official template. At this point, the proposal champion is free to move on to the next stage. If a champion doesn't exist yet, then an accepted proposal may remain open until a champion volunteers by posting in the GitHub Issue. | ||
When a proposal is accepted, a TSC member will create a new GitHub Issue summarizing the original proposal using our official template. |
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 think we should change this paragraph and say that a "TSC member might delegate this task to the champion of the proposal if there's one already.". This will actually reflect what we have been doing recently.
When a proposal is accepted, a TSC member will create a new GitHub Issue summarizing the original proposal using our official template. At this point, the proposal champion is free to move on to the next stage. If a champion doesn't exist yet, then an accepted proposal may remain open until a champion volunteers by posting in the GitHub Issue. | ||
When a proposal is accepted, a TSC member will create a new GitHub Issue summarizing the original proposal using our official template. | ||
|
||
The proposal's original author will be given a chance to volunteer to be its champion. If they refuse, someone else can volunteer for the position by posting in the GitHub Issue. An accepted proposal will remain an open issue until a champion is confirmed by TSC. Once confirmed, the champion is free to advance the proposal to the next stage. |
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 would be more precise and say that a champion "is confirmed" when the Stage 2 template is updated to reflect this.
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.
Currently a champion is not required to reach stage 2 and the text says this. Are you suggesting that we change it so that there is one?
I could see wanting to do that, but the downside is some times a champion can't work on something right away, for example @natemoo-re proposal for the Container API. Maybe that was a past mistake we can avoid in the future though.
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, I see now; I misunderstood things. However, this part is confusing:
Once confirmed, the champion is free to advance the proposal to the next stage.
Does that mean that Stage 2 proposals move forward only when they have a champion, meaning that the TSC doesn't need to be involved anymore?
@@ -26,13 +26,15 @@ | |||
|
|||
**Goal:** Confirm proposal feasibility with Astro Maintainers and the [Technical Steering Committee (TSC)](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#technical-steering-committee-tsc). | |||
|
|||
**Requirements:** An existing proposal (Stage 1). In addition, a proposal is more likely to be accepted if it is detailed and well thought-out, can demonstrate community interest, has at least one champion volunteer, and has buy-in/interest from Astro maintainer(s). | |||
**Requirements:** An existing proposal (Stage 1). In addition, a proposal is more likely to be accepted if it is detailed and well thought-out, can demonstrate community interest, has at least one champion volunteer, has a maintainer acting as a champion, and has buy-in/interest from Astro maintainer(s). |
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.
It could be a nitpick, but here we use the term maintainer (lower case), but it's a generic term and a bit foggy. Who's an Astro maintainer? Did we want to use the L2 level from the governance?
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.
Yes, I did mean L2 maintainer. Should I put that in here?
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 think so, at least the first time we use the word
2 points of clarification: