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

Clarify RFC meetings #904

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Clarify RFC meetings #904

wants to merge 1 commit into from

Conversation

matthewp
Copy link
Contributor

The process calls for RFC meetings, which we have not had formally in quite a while. This change brings those back and clarifies who can call for them.

Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @matthewp ! A couple of grammar nits, and some questions for clarity that we might also want to include in the text.

@@ -30,7 +30,7 @@

**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.
**What to Expect:** A proposal reaches this stage (aka "is accepted") during an [#rfc-meetings](RFC Meeting) with Maintainers and TSC, following our existing [RFC voting](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) process.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
**What to Expect:** A proposal reaches this stage (aka "is accepted") during an [#rfc-meetings](RFC Meeting) with Maintainers and TSC, following our existing [RFC voting](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) process.
**What to Expect:** A proposal reaches this stage (aka "is accepted") during an [RFC Meeting](#rfc-meetings) with Maintainers and TSC, following our existing [RFC voting](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) process.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does "Maintainers" include L2 (maintainers) and L3 (core)? I think we should be precise

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going with what Matthew said yesterday in Community Call it refers to >= L2, but I agree we should be precise here.


RFCs advance through the stages during RFC meetings with TSC and maintainers. Voting follows the [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) voting process.

Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels it is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels it is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.
Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels the proposal is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.

"contacting TSC" is pretty vague. Unless we describe how to do that elsewhere on this page, I'd specify how they should contact TSC. Do they mention the role in the proposal? Email?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Sarah that we should specify the medium of how TSC should be contacted.

I would remove the "proposal author" from the equation because it can be anyone. We take for granted that "proposal authors" are maintainers and core, but I think it's not like that. "Proposal authors" can be anyone.

Instead, I would reword this by just mentioning "champion" and specifying that a champion can be the proposal author, but not necessary.


Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels it is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.

All maintainers are invited to the meeting. If consensus is reached the RFC advances to the next stage. See [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) documentation for full details.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
All maintainers are invited to the meeting. If consensus is reached the RFC advances to the next stage. See [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) documentation for full details.
All maintainers are invited to the meeting. If consensus is reached, the RFC advances to the next stage. See [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) documentation for full details.

Should we also say what happens if consensus is not reached? Is there a waiting limit before asking again? Is the proposal officially dead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The governance document talks about that some: https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals

Was trying not to repeat too much here that's already covered in the governance doc.


RFCs advance through the stages during RFC meetings with TSC and maintainers. Voting follows the [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) voting process.

Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels it is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would doing these discussions during a Tuesday community call actually be a good idea? Would it increase participation/attendance (esp. from the rest of core), give @sarah11918 a built-in agenda item, and require less planning to organize and find a time?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Especially reading the next paragraph, it feels a little "secret meeting" and the transparency wins of doing it on a public call could be worth it for that alone.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could, but some issues we had before when RFC meetings were open to all:

  • Discussions tended to be very high-level, most of the audience didn't understand the technical details that are an important part.
  • Discussions tended to veer towards bikeshedding often.
  • It was always odd to me that we tried to reach consensus to the general community, which meant a non-maintainer could object and per the rules it then went to a TSC decision.

Copy link
Member

@ematipico ematipico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new changes often use the word "maintainers" (lower case), although it isn't clear who they are. L2 + L3? Only L2? A generic term to specify anyone who contributes to Astro? I think we should be more precise.

@@ -30,7 +30,7 @@

**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.
**What to Expect:** A proposal reaches this stage (aka "is accepted") during an [#rfc-meetings](RFC Meeting) with Maintainers and TSC, following our existing [RFC voting](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) process.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does "Maintainers" include L2 (maintainers) and L3 (core)? I think we should be precise


RFCs advance through the stages during RFC meetings with TSC and maintainers. Voting follows the [RFC proposal](https://github.com/withastro/.github/blob/main/GOVERNANCE.md#voting-rfc-proposals) voting process.

Meetings occur ad hoc rather than on a scheduled basis. They are called for when a proposal author or champion feels it is ready to advance to the next stage. The author or champion can ask for a meeting by contacting TSC to schedule a time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Sarah that we should specify the medium of how TSC should be contacted.

I would remove the "proposal author" from the equation because it can be anyone. We take for granted that "proposal authors" are maintainers and core, but I think it's not like that. "Proposal authors" can be anyone.

Instead, I would reword this by just mentioning "champion" and specifying that a champion can be the proposal author, but not necessary.

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.

None yet

5 participants