如何在Gerrit上审查一个合并请求

427 阅读2分钟

在这篇文章中,我们已经解释了在Gerrit上审查合并请求的详细步骤。代码审查是软件开发中的一个重要步骤。

前提是:

审查Gerrit上的合并请求

要在Gerrit上审查一个拉动请求,你可以进入项目页面并浏览被修改的文件。

点击一个行号,将打开一个对话框,你可以在那里添加你的评论。一旦你添加了所有的评论,你需要通过点击项目页面顶部的 "回复 "按钮来提交它。

你需要提交两个分数:

  • 代码审查得分
  • 代码验证得分

代码审查的不同分数的含义如下:

  • -2: 这个拉动请求不应该被合并
  • -1: 该拉动请求中的某些修改将被优先考虑
  • 0: 没有意见
  • 1: 看起来不错,但必须有其他人批准
  • 2: 看起来不错,可以合并

Code-Verify的不同分数的含义如下:

  • -1: 代码没有按照预期工作
  • 0:我没有通过运行代码来验证它
  • 1: 代码按预期运行

你需要检查的事项

你需要检查代码-审查得分的事情:

  • 按照代码质量标准检查代码的质量
  • 检查是否涵盖了所有的边缘情况
  • 检查代码是否有良好的注释

在审查完代码后,检查这些代码的验证分数:

  • 在你的系统上下载补丁,检查它是否按预期运行
  • 你可能需要检查它是否干扰了其他功能

这将打开另一个对话框,你可以添加一些注释,添加你的代码审查分数并提交。

其他审查者和打开拉动请求的开发者会收到一封邮件通知。

通过OpenGenus的这篇文章,你一定对如何在Gerrit上审查合并请求有了一个完整的概念。

享受在Gerrit上审查拉动请求的乐趣吧。