在这篇文章中,我们已经解释了在Gerrit上审查合并请求的详细步骤。代码审查是软件开发中的一个重要步骤。
前提是:
审查Gerrit上的合并请求
要在Gerrit上审查一个拉动请求,你可以进入项目页面并浏览被修改的文件。
点击一个行号,将打开一个对话框,你可以在那里添加你的评论。一旦你添加了所有的评论,你需要通过点击项目页面顶部的 "回复 "按钮来提交它。
你需要提交两个分数:
- 代码审查得分
- 代码验证得分
代码审查的不同分数的含义如下:
- -2: 这个拉动请求不应该被合并
- -1: 该拉动请求中的某些修改将被优先考虑
- 0: 没有意见
- 1: 看起来不错,但必须有其他人批准
- 2: 看起来不错,可以合并
Code-Verify的不同分数的含义如下:
- -1: 代码没有按照预期工作
- 0:我没有通过运行代码来验证它
- 1: 代码按预期运行
你需要检查的事项
你需要检查代码-审查得分的事情:
- 按照代码质量标准检查代码的质量
- 检查是否涵盖了所有的边缘情况
- 检查代码是否有良好的注释
在审查完代码后,检查这些代码的验证分数:
- 在你的系统上下载补丁,检查它是否按预期运行
- 你可能需要检查它是否干扰了其他功能
这将打开另一个对话框,你可以添加一些注释,添加你的代码审查分数并提交。
其他审查者和打开拉动请求的开发者会收到一封邮件通知。
通过OpenGenus的这篇文章,你一定对如何在Gerrit上审查合并请求有了一个完整的概念。
享受在Gerrit上审查拉动请求的乐趣吧。