本文档提供有关如何审核变更的说明。请注意以下有效变更的标准
- 此变更是否符合此项目的既定目标?
- 此变更是否在此项目现有架构内有效?
- 此变更是否会引入可能在未来导致问题的功能缺陷?
- 此变更是否遵循为此项目建立的最佳实践?
- 此变更是否是执行所述功能的良好方式?
- 此变更是否会引入任何安全风险或不稳定性风险,或者破坏任何现有应用?
如需了解有关审核变更的其他信息,请参阅 Gerrit 文档的审核变更部分。
如果您被指定为变更的审核者,您应该会收到一封包含变更链接的电子邮件。请按照以下步骤审核和复审变更
点击电子邮件中变更的链接。此时会显示 Gerrit 的“变更信息”屏幕。此屏幕提供变更的描述,指明变更的贡献者和所有审核者,并提供与变更关联的文件列表。
阅读变更描述,了解变更的目标。
点击文件以审核文件中的更改。文件会在双窗格比较屏幕中打开,左侧显示现有文件,右侧显示包含建议更改的文件。要审核文件,请执行以下操作
使用鼠标突出显示(点击 + 拖动)您要评论或提问的代码部分。此时会显示消息“按 c 键进行评论”。
按 c 键。此时会显示草稿评论框。
输入您的评论或问题,然后点击保存。
点击变更的唯一编号(屏幕左上角),返回到“变更信息”屏幕。
继续审核变更中的每个文件。
当您完成审核变更中的所有文件后,返回到“变更信息”屏幕,然后点击回复。此时会显示“提交要求投票”屏幕。
为变更选择以下评分之一
- +1 我觉得不错,但还需要其他人批准。
0 无评分
-1 我希望不要按原样提交此版本,有些地方需要修复。
-2 不应提交此变更
点击回复,将您的评论和评分发送给变更的开发者。
如果存在需要修复的问题,开发者会修复这些问题,并要求您复审修复。否则,您就完成了变更审核。