对更改进行审核

本文档提供了有关如何审核更改的说明。请注意,有效更改须满足以下条件:

  • 这项更改是否符合此项目的既定目的?
  • 这项更改在项目的现有架构中是否有效?
  • 这项更改是否会引入可能在将来造成问题的设计缺陷?
  • 这项更改是否遵循了针对此项目制定的最佳做法?
  • 这项更改是否是执行所述功能的绝佳方式?
  • 这项更改是否会带来任何安全风险或不稳定性方面的风险,或者破坏任何现有的应用?

如需了解如何审核更改,请参阅 Gerrit 文档的对更改进行审核部分。

如果您被指定为某项更改的审核人员,则应该会收到一封电子邮件,其中包含指向该项更改的链接。请按照以下步骤查看并审核更改:

  1. 点击电子邮件中指向相应更改的链接。此时将显示 Gerrit 的“更改信息”屏幕。此屏幕提供更改说明,标识出更改的贡献者和所有审核人员,并提供与更改相关的文件列表。

  2. 阅读更改描述以了解更改的目标。

  3. 点击某个文件即可查看文件中的更改。该文件在双窗格比较屏幕中打开,左侧显示现有文件,右侧显示包含建议更改的文件。如需审核文件,请执行以下操作:

    1. 使用鼠标突出显示(点击 + 拖动)您想要发表评论或提出疑问的代码部分。此时将显示“按 c 键进行评论”消息。

    2. c 键。此时将显示草稿评论框。

    3. 输入您的评论或问题,然后点击保存

  4. 点击更改的唯一编号(屏幕左上角)返回“更改信息”屏幕。

  5. 继续审核更改所涉及的每个文件。

  6. 审核完更改所涉及的所有文件后,返回“更改信息”屏幕并点击回复。此时将显示“提交要求投票”屏幕。

  7. 为更改选择以下评分之一:

    • +1 看起来不错,但还需要其他人批准。
    • 0 没有评分

    • -1 我希望不要按原样提交,因为有些问题需要修复。

    • -2 不应提交这项更改

  8. 点击回复,将您的评论和评分发送给更改的开发者。

如果有问题需要修复,开发者会修复它们并请您审核修复内容。如果没有,那么您已完成对更改的审核。