跳到主要内容

V8提交者和评审者的职责

当您向V8代码库提交时，请确保遵循以下指南（改编自https://dev.chromium.org/developers/committers-responsibility）：

  1. 找到适合的评审者来审核您的更改，以及您被要求审核的补丁。
  2. 在提交更改之前和之后保持IM和/或电子邮件的可达性。
  3. 检查流水线，直到您的更改使所有机器人都变为绿色状态。
  4. 当提交一个TBR更改（待评审）时，请确保通知您更改代码的相关人员。通常只需发送评审电子邮件即可。

简而言之，请为项目做正确的事情，而不是为了让代码被提交而选择最简单的方法，最重要的是：使用您的最佳判断能力。

不要害怕提问，总有会立即查看发送到v8-committers邮件列表的人可以帮助您。

含有多个评审者的更改​

有时会遇到需要多个评审者的更改，因为有些更改涉及多领域的责任和专业知识，因此需要多个人的关注。

问题在于，如果没有一些指导，就没有明确的责任分配给这些评审。

如果您是某个更改的唯一评审者，您知道自己必须认真完成评审工作。而当有三个其他人参与时，您有时可能会假设其他人已经仔细检查了评审的某些部分。有时所有评审者都会这样认为，导致更改未被正确评审。

在其他情况下，一些评审者对补丁表示“LGTM”，而另一些人仍在等待进一步的更改。这会让作者对评审状态感到困惑，甚至有些补丁在未获得所有评审者认可之前被提交。

与此同时，我们希望鼓励更多人参与评审流程，关注正在发生的事情。

以下是一些指导进程的建议：

  1. 当补丁作者请求多个评审者时，他们应在评审请求电子邮件中明确表明每个评审者的职责。例如，您可以在电子邮件中写道：

    - larry: 位图更改
    - sergey: 处理漏洞
    - 其他人：仅供参考

  2. 在这种情况下，您可能因为希望了解多进程更改而在评审名单中，但您并非主要评审者，作者和其他评审者也不会期望您详细审核所有差异。

  3. 如果您收到包含许多其他人的评审请求，而作者未执行第(1)条，请询问他们您负责的部分，以免进行详细全面的评审。

  4. 作者应在所有评审列表中的人员批准后再提交。

  5. 参与评审但没有明确评审责任的人（即简单的顺手评审）应快速响应，不应延误评审。补丁作者若遇到响应迟缓的，可以毫不留情地追问。

  6. 如果您是某个评审的“仅供参考”人员，并且实际上没有详细评审（甚至根本没有评审），但对补丁没有异议，请标注出来。您可以使用诸如“橡皮图章”或“ACK”之类的词，而不是“LGTM”。这样，真正的评审者知道他们不能指望您完成他们的工作，而补丁的作者知道他们无需等待您的进一步反馈。希望我们仍然能让所有人参与其中，同时有明确的责任分配和详细评审。这甚至可能加速某些更改，因为您可以快速对不关心的更改“ACK”，作者知道他们无需等待您的反馈。