本文档提供了有关如何审核更改的说明。请注意,有效更改须满足以下条件:
- 这项更改是否符合此项目的既定目的?
- 这项更改在项目的现有架构中是否有效?
- 这项更改是否会引入可能在将来造成问题的设计缺陷?
- 这项更改是否遵循了针对此项目制定的最佳做法?
- 这项更改是否是执行所述功能的绝佳方式?
- 这项更改是否会带来任何安全风险或不稳定性方面的风险,或者破坏任何现有的应用?
如需了解如何审核更改,请参阅 Gerrit 文档的对更改进行审核部分。
如果您被指定为某项更改的审核人员,则应该会收到一封电子邮件,其中包含指向该项更改的链接。请按照以下步骤查看并审核更改:
点击电子邮件中指向相应更改的链接。此时将显示 Gerrit 的“更改信息”屏幕。此屏幕提供更改说明,标识出更改的贡献者和所有审核人员,并提供与更改相关的文件列表。
阅读更改描述以了解更改的目标。
点击某个文件即可查看文件中的更改。该文件在双窗格比较屏幕中打开,左侧显示现有文件,右侧显示包含建议更改的文件。如需审核文件,请执行以下操作:
使用鼠标突出显示(点击 + 拖动)您想要发表评论或提出疑问的代码部分。此时将显示“按 c 键进行评论”消息。
按 c 键。此时将显示草稿评论框。
输入您的评论或问题,然后点击保存。
点击更改的唯一编号(屏幕左上角)返回“更改信息”屏幕。
继续审核更改所涉及的每个文件。
审核完更改所涉及的所有文件后,返回“更改信息”屏幕并点击回复。此时将显示“提交要求投票”屏幕。
为更改选择以下评分之一:
- +1 看起来不错,但还需要其他人批准。
0 没有评分
-1 我希望不要按原样提交,因为有些问题需要修复。
-2 不应提交这项更改
点击回复,将您的评论和评分发送给更改的开发者。
如果有问题需要修复,开发者会修复它们并请您审核修复内容。如果没有,那么您已完成对更改的审核。