-
Notifications
You must be signed in to change notification settings - Fork 15k
[QualGroup][docs] Document decision-taking process and refresh sync-up materials #163122
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
[QualGroup][docs] Document decision-taking process and refresh sync-up materials #163122
Conversation
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.
Though, I would like to suggest an edition:
In "Current Topics & Backlog", move the last item in the list
- (add future entries here)
to the first position, as the entries are being added in reverse order.
|
@uwendi Thank you for your feedback — I’ve addressed the review comments. |
Looks perfect, thank you. |
|
I noticed a couple of things that should be addressed before this can be merged: 1. Merge commits LLVM follows a linear-history policy, which means merge commits are not permitted. As stated in the LLVM Developer Policy (https://llvm.org/docs/Contributing.html#for-developers-to-commit-changes-from-git): "LLVM has a linear-history policy, which means that merge commits are not allowed, and the main branch is configured to reject pushes that include merges." Please rebase your branch on top of the latest main instead of merging it: This will remove the merge commits and produce a clean, linear history consistent with LLVM’s workflow. 2. Broken hyperlink in the PR description Finally, please take a moment to review the full LLVM Contributing Guide (https://llvm.org/docs/Contributing.html) — it contains useful details about commit structure, review conventions, and contribution etiquette. |
…p materials * Introduce a new “Decision Taking” section describing principles, consensus-first flow, time limits, voting procedure, and documentation of outcomes. * Update Meeting Materials with the October 2025 presentation slides. This documents how the WG makes decisions and keeps contributor resources current.
* Sort slide links from newest to oldest, keeping the placeholder on top.
41d4b21 to
28d7e03
Compare
|
@petbernt Thank you for your feedback. |
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 for this contribution.
Looks good to me.
llvm/docs/QualGroup.rst
Outdated
| To prevent open-ended debates, if no new viewpoints are expressed after an agreed period (e.g., 2 weeks), the chair (or moderator) may: | ||
|
|
||
| * **Summarize the apparent consensus** and close the discussion, or | ||
| * **Postpone the topic** to the next sync-up meeting if the outcome remains unclear, or | ||
| * **Call for a short vote** to confirm the group’s position. |
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 clarify who the chair/moderator is. If there isn't an official one, maybe we can make person who starts the thread should be responsible for taking one of these three actions if the discussion stalls for the agreed-upon period ?
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.
Good point — I agree that clarifying who the chair/moderator is makes sense.
Based on Wendi’s suggestion, I’d propose adding the following clarification after the current line:
To prevent open-ended debates, if no new viewpoints are expressed after an agreed period (e.g., 2 weeks), the chair (or moderator) may:
+ The chair (typically the person who started the Discord or Discourse thread) acts as the moderator, responsible for summarizing or escalating the discussion as appropriate.
This addition, reflecting Wendi’s suggestion, makes the transition smoother and defines the temporary chair role more clearly.
Also, this is a temporary definition — later, once we designate a chair and clarify their role and term as Wendi suggested, we can revise it accordingly.
@uwendi @ZakyHermawan What do you think?
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.
@IamYJLee
Thanks, that’s a very good proposal. I really like how you phrased it as a temporary clarification.
My opinion: perhaps we can refer to “the moderator (typically the person who started the thread)” instead of “the chair (typically the person who started the thread)”. This keeps the language consistent with our current practice, where moderation happens ad hoc and organically, and no formal chair role exists yet.
Using moderator makes it clearer that this person’s role is simply to guide the discussion and summarize consensus, not to make decisions on behalf of the group. If we later define a chair or co-chairs (for coordination and continuity), we can update this section accordingly.
The only thing we temporarily lose by completely removing “the chair” (which was never formally designated anyway) is a clear external point of contact and a bit of continuity across discussions. For now, though, these can be handled collectively until the group chooses to formalize that role, if it ever feels necessary.
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.
|
@ZakyHermawan Naming a chair too early can appear hierarchical. In LLVM (and in open source more broadly), we tend to prefer shared stewardship, so the chair’s role should remain facilitative, not authoritative. In a working group, the chair typically:
For reference, a comparable role exists in other open-source safety initiatives. For example, Philipp Ahmann serves as chair of the ELISA Lighthouse OSS WG, where the position is mainly organizational, not directive. In our context, a chair could be designated now, based on the general opinion of the core members, but we should then add a short clarification about its role in the documentation. The role could also rotate among active contributors in the future. Until then, following your suggestion, the person who initiates a Discord (or Discourse) thread would act as the moderator, taking responsibility for summarizing or escalating the discussion after the agreed period. What do you think? |
|
By the way, I forgot to mention that who becomes the chair depends on the context:
In the example I mentioned from the Lighthouse OSS Working Group (ELISA Project), which is a community-driven group, Philipp is the person who initiated the group, with Gabriele serving as co-chair. |
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.
After review, I only have two minor edition requests.
llvm/docs/QualGroup.rst
Outdated
|
|
||
| When consensus cannot be reached or when a clear yes/no decision is needed: | ||
|
|
||
| * The chair (or moderator) may call for a **simple vote** using emoji reactions on Discord or a similar visible method. |
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.
As you introduced the definition of moderator in "Consensus and Time Limits", I suggest to completely remove "the chair" from this line:
- The moderator may call for a simple vote using emoji reactions on Discord or a similar visible method.
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.
llvm/docs/QualGroup.rst
Outdated
|
|
||
| Discussions remain open until a clear consensus emerges, meaning no sustained objections have been raised after reasonable discussion. | ||
|
|
||
| To prevent open-ended debates, if no new viewpoints are expressed after an agreed period (e.g., 2 weeks), the moderator (typically the person who started the thread) may take one of the following actions: |
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.
Just a small edition here:
<...> the moderator (typically the person who started the discussion thread) may take one <...>
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.
Please consider the changes suggested by Wendi. That aside, it looks good to me.
…ify reference to 'discussion thread'.
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
|
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.
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.
LGTM
…p materials (llvm#163122) ## Summary This PR documents the LLVM Qualification Working Group’s decision-taking process and updates the latest sync-up meeting materials. ## Changes * Added a new **Decision Taking** section describing: * Principles (consensus-first, inclusiveness, transparency)Discussion time limits * Voting procedure * Documentation of outcomes * Updated Meeting Materials with the October 2025 presentation slides. ## Background The LLVM Qualification Working Group aims to ensure decisions are made transparently and collaboratively. This new section documents how proposals are discussed, consensus is reached, and outcomes are recorded, providing clear guidance for contributors. It also keeps the group’s meeting resources up to date for easier reference and continuity. ## Testing - [x] Documentation builds successfully - [x] New “Decision Taking” section renders correctly - [x] Links and internal references are properly formatted - [x] Meeting Materials section updated with the October 2025 slides ## Related Links * [LLVM Qualification Working Group Documentation](https://llvm.org/docs/QualGroup.html)
…p materials (llvm#163122) ## Summary This PR documents the LLVM Qualification Working Group’s decision-taking process and updates the latest sync-up meeting materials. ## Changes * Added a new **Decision Taking** section describing: * Principles (consensus-first, inclusiveness, transparency)Discussion time limits * Voting procedure * Documentation of outcomes * Updated Meeting Materials with the October 2025 presentation slides. ## Background The LLVM Qualification Working Group aims to ensure decisions are made transparently and collaboratively. This new section documents how proposals are discussed, consensus is reached, and outcomes are recorded, providing clear guidance for contributors. It also keeps the group’s meeting resources up to date for easier reference and continuity. ## Testing - [x] Documentation builds successfully - [x] New “Decision Taking” section renders correctly - [x] Links and internal references are properly formatted - [x] Meeting Materials section updated with the October 2025 slides ## Related Links * [LLVM Qualification Working Group Documentation](https://llvm.org/docs/QualGroup.html)
Summary
This PR documents the LLVM Qualification Working Group’s decision-taking process and updates the latest sync-up meeting materials.
Changes
Added a new Decision Taking section describing:
Updated Meeting Materials with the October 2025 presentation slides.
Background
The LLVM Qualification Working Group aims to ensure decisions are made transparently and collaboratively.
This new section documents how proposals are discussed, consensus is reached, and outcomes are recorded, providing clear guidance for contributors.
It also keeps the group’s meeting resources up to date for easier reference and continuity.
Testing
Related Links