V8 提交者和审阅者的职责

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

  1. 为您的更改和您被要求审阅的补丁找到合适的审阅者。
  2. 在您提交更改之前和之后，请通过 IM 和/或电子邮件保持联系。
  3. 在您的更改之后，请关注 瀑布，直到所有机器人变为绿色。
  4. 在提交 TBR（待审阅）更改时，请确保通知您更改了代码的人员。通常只需发送审阅电子邮件即可。

简而言之，为项目做正确的事，而不是最容易提交代码的事，最重要的是：运用您的最佳判断。

不要害怕提出问题。总有人会立即阅读发送到 v8-committers 邮件列表的消息，他们可以帮助您。

具有多个审阅者的更改 #

偶尔会有很多审阅者的更改，因为有时由于多个责任领域和专业知识，可能需要几个人参与更改。

问题在于，如果没有一些准则，这些审阅中就没有明确的责任分配。

如果您是更改的唯一审阅者，您知道您必须做好工作。当有三个其他人时，您有时会认为其他人一定仔细查看了审阅的某些部分。有时所有审阅者都这样认为，更改没有得到适当的审阅。

在其他情况下，一些审阅者对补丁说“LGTM”，而另一些审阅者仍在期待更改。作者可能会对审阅的状态感到困惑，并且一些补丁在至少一名审阅者期望在提交之前进行进一步更改的情况下被签入。

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

因此，这里有一些准则可以帮助澄清流程

  1. 当补丁作者请求多个审阅者时，他们应该在审阅请求电子邮件中明确说明他们对每个审阅者的责任期望。例如，您可以在电子邮件中写下以下内容

    - larry: bitmap changes
- sergey: process hacks
- everybody else: FYI

  2. 在这种情况下，您可能在审阅列表中，因为您要求参与多进程更改的循环，但您不是主要审阅者，作者和其他审阅者不会期望您详细审阅所有差异。

  3. 如果您收到包含许多其他人的审阅，并且作者没有执行 (1)，如果您不想详细审阅整个内容，请询问他们您负责的哪一部分。

  4. 作者应该在签入之前等待审阅列表中所有人的批准。

  5. 那些没有明确审阅责任的审阅者（即路过审阅）应该超级响应，不要阻碍审阅。如果他们这样做，补丁作者可以随意无情地提醒他们。

  6. 如果您是审阅中的“FYI”人员，并且您没有真正详细审阅（或根本没有审阅），但对补丁没有问题，请注意这一点。您可以说“rubber stamp”或“ACK”而不是“LGTM”。这样，真正的审阅者就知道不要相信您为他们做了他们的工作，但补丁的作者知道他们不必等待您的进一步反馈。希望我们仍然可以将所有人纳入循环，但拥有明确的归属权和详细的审阅。它甚至可能加快一些更改，因为您可以快速“ACK”您不关心的更改，并且作者知道他们不必等待您的反馈。