CommitFest 清单

CommitFest 清单和时间线

此页面包含 CommitFest 管理员 (CFM) 需要做的事情以及何时做的事情的清单。 (目前正在不断变化，正在重新编写以反映现代实践。)

志愿者

在 CommitFest 开始前的几周（在快速检查该职位是否已有人担任后），向 pgsql-hackers 发送电子邮件，自愿担任 CFM 职位。 基本上是先到先得。

CommitFest 开始前三到四周

幸运的是，你至少在这之前就知道自己将会是 CommitFest 管理员。 如果是，请执行以下操作

• 尝试从 -hackers 和其他地方招募一名助理 CFM。 这是可选的，但如果你对该职位不熟悉，或者 CommitFest 规模很大，则建议这样做。
• 如果你没有定期这样做，请开始将所有 pgsql-hackers 邮件保存在本地邮件存储中以供搜索。
• 检查你在 CommitFest 上是否有管理权限。（你应该在顶部栏中看到一个“管理”链接。）如果没有，请向 pgsql-www 请求权限修复。

CommitFest 开始前 5 到 7 天

• 在 pgsql-hackers（以及你的博客？）上发布一个提醒，告知 CommitFest 将在日期开始，并提醒他们应该将他们的补丁注册/提交。
• 进行补丁扫描（见下文）。
• 查看待处理的补丁并为其分配类别，或者如果类别错误，则更改其类别。
• 检查 CommitFest 中的每一个补丁，并查找/更新
  • 标记为“等待作者”的补丁，如果作者已回复反馈，则将其状态更改为“需要审阅”。
  • 已在 CommitFest 开始之前提交/拒绝/退回的补丁，并相应地更新其状态。
  • 根据 cfbot CI 不再适用的补丁，提醒作者在 CommitFest 开始之前发送重新基准化的版本。

补丁扫描：搜索 pgsql-hackers 存档中包含补丁附件的邮件，这些邮件是在上次 CommitFest 开始后发送的。（这通常是两个月的范围，但有时长达四个。）。确保所有提交到 -hackers 的补丁都包含在 CommitFest 中（或上一个 CommitFest 中）。 如果没有，请发邮件给补丁作者，询问他们是否希望将其补丁提交到 CommitFest（见下面的模板）。 新贡献者可能应该添加到 CommitFest 中，即使他们回复较慢，但资深贡献者则不应添加。

CommitFest 开始前 2 天

• 向 -hackers 发送提醒，告知 CommitFest 将在 2 天后开始。

CommitFest 第一天

• 将 CommitFest 状态从“打开”更改为“正在进行中”。
• 如果需要，更改下一个 CommitFest 的状态。
• 在 -hackers 上宣布 CommitFest 已经开始，任何新的补丁都需要放在下一个 CommitFest 中。
• 向所有补丁提交者发送提醒，告知他们在 CommitFest 中有一个补丁，并且预计他们会审阅其他人提交的复杂度相当的补丁。

CommitFest 每 5 到 7 天

• 检查标记为“等待作者”的补丁。
  • 如果作者已回复反馈并再次等待审阅，请将状态更改为“需要审阅”。
• 检查标记为“等待审阅”且已分配审阅者的补丁。
  • 如果审阅已完成并发现需要修复的内容，并且作者尚未回复，请将补丁标记为“等待作者”。 对于非常大的补丁，你可能希望等待的时间比 5-7 天长一些。
  • 如果审阅者说它“已准备好提交”，请将其更改为该状态。
  • 如果之前的讨论似乎已经停止，请考虑在列表中请求状态更新。
• 检查标记为“已准备好提交”的补丁。
  • 检查这些补丁的 cfbot。 如果存在提交者无法在收到补丁时应用补丁集的风险，请考虑提醒需要重新基准化的补丁集的作者。（如果作者也是提交者，则可能不需要此提醒。）

CommitFest 结束前 5 到 7 天

• 对于所有打开的补丁
  • 如果没有最近的讨论，请向审阅者发送电子邮件，提醒他们 CommitFest 即将结束，并询问他们是否进展顺利，或者是否需要帮助。
  • 如果有最近的讨论，请在讨论线程中发布，询问补丁是否可能在 CommitFest 结束之前准备就绪。

CommitFest 结束前 3 天

• 在 -hackers 上发布，提醒 CommitFest 最后一天将对打开的补丁采取什么措施。

CommitFest 最后一天

• 对于标记为“等待作者”且至少有一次审阅的补丁，将其设置为“返回反馈”并发送相应的电子邮件。
• 对于标记为“需要审阅”的补丁
  • 如果它至少收到一次良好的审阅，请将其移至下一个 CommitFest（移除当前的审阅者预留）。
  • 否则，请将其保留为待处理状态。

突然死亡加时赛

CommitFest 结束日期过后，补丁的处理方式会发生变化。

• 一旦补丁获得可靠的审阅，如果它需要任何更改，它将被标记为“返回反馈”。
• 标记为“已准备好提交”但最终需要进一步工作的补丁也将“返回”，而不是进入“等待作者”状态。
• 需要进一步规格讨论、性能测试等的补丁将立即“返回”。
• 来自尚未进行任何审阅的提交者的补丁，并且仍然处于“需要审阅”状态的补丁将被推迟到下一个 CommitFest。

目标是在尽可能短的时间内结束 CommitFest，而不会跳过任何仍然需要审阅的补丁的审阅。

CommitFest 后

• 在应用程序中关闭 CommitFest（但要确保所有补丁都先移动；否则，像 cfbot 这样的构建机器会将其删除）。
• 使下一个 CommitFest 成为当前 CommitFest。
• 向有返回反馈补丁的作者发送提醒邮件，提醒他们如果计划继续处理补丁，请将其添加到下一个 CommitFest。
• 向审阅者发送“感谢”邮件，提醒他们即使他们没有提交补丁，也欢迎他们在下一个 CommitFest 中进行审阅。
• 为最终的发行说明组装两个审阅者列表。
  • 对补丁进行大量工作（通常是代码工作）以使其生效的审阅者以及他们所处理的补丁，以及
  • 进行常规审阅的审阅者，作为一大组。
• 喝一两杯啤酒 - 你应得的。 :-)

邮件模板

补丁扫描

对于新贡献者

   [contributor name]:

   You sent in your patch, [name of patch file] to pgsql-hackers on [date posted].  
   You may not be aware, but in order for your patch to be included in PostgreSQL, 
   it needs to be part of the next CommitFest.  Accordingly, I have added your 
   patch to the list here [link to CF entry].  If you want to withdraw your patch, 
   or if you simply won't be around to answer questions during the 30 days of 
   this CommitFest, please let me know and I'll remove it.  Thanks!

对于有经验的贡献者

   [contributor name]:

   You sent in your patch, [name of patch file] to pgsql-hackers on [date posted], 
   but you did not post it to the next CommitFest [link to CF page].  If this was 
   intentional, then you need take no action.  However, if you want your patch to 
   be reviewed as part of the upcoming CommitFest, then you need to add it yourself
   before [start date].  Thanks for your contributions.

补丁提醒

   [contributor name]:

   As a reminder, you have the following patches in the current CommitFest:

   [list of links to their patches]

   If you are not prepared to follow up on review to these patches during this 
   CommitFest, please let me know and I will remove them now.  I also remind
   you that each patch submitter is expected to review at least one patch from
   another submitter during the CommitFest, so please pick someone else's
   patch to review as soon as you can.  

   Thanks for your contributions to PostgreSQL!

审阅者已完成

   [reviewer name]:

   Since you haven't had time to write a review of [patch] in the last 5 days, 
   we've taken your name off the reviewer list for this patch.  Of course, you 
   are still very welcome to review it if you can find time.  We're just removing 
   your name so that other reviewers know the patch still needs attention.  
   We understand that day jobs and other things get in the way of doing patch review 
   when you want to, so please come back and review a patch or two later 
   when you have more time.  

   Thanks!

返回反馈

   [author name]:

   [patch], which you submitted to this CommitFest, has been awaiting your attention 
   for more than five days.  As such, we have moved it to "Returned with Feedback" 
   and removed it from the reviewing queue.  Depending on timing, this may be reversable, 
   so let us know if there are extenuating circumstances.  In any case, you are welcome 
   to address the feedback you have received, and resubmit the patch to the next CommitFest.  

   Thank you for contributing to PostgreSQL.

未审阅提交者列表

   Subject: Submitters who have not reviewed

   Per the Developer Meeting of 2012, each patch submitter is required 
   to do at least one patch review for each submitted patch.  The following
   people have submitted one or more patches to this CommitFest, and have 
   neither claimed any patch for review, nor responded to a private reminder
   to do so.  This list is being posted in hopes that these submitters will 
   see it and do a review soon.  Without each submitter doing a review, 
   the PostgreSQL project cannot promise to review all patches.

   [list of names]