1. 产品简介
  2. 快速开始
  3. 编写构建流程
  4. 配置构建计划
  5. 构建环境依赖包
  6. 构建制品
  7. 构建节点
  8. 管理构建计划
  9. 系统插件
  10. 自定义团队插件
  11. 最佳实践
  12. 常见问题
  13. 词汇表
代码仓库 / 合并请求与代码评审 / 评审合并请求

评审合并请求

在新建合并请求或合并请求合并前,开发者可更新标题和描述,添加其他项目成员为评审者,并关联项目内资源(如任务、文件、合并请求、Wiki)。你还可以通过 CI 插件自动添加评审者

如果合并请求的目标分支为保护分支,且为保护分支配置了管理员并开启「自动添加分支管理员为评审者」,对应授权数量的分支管理员会自动添加为评审者。

评审内容

评审者收到邀请评审通知后,对代码进行评审的内容一般会聚焦于以下两点:

  • 该合并请求的标题、描述及关联资源是否对代码改动作出充分说明,评审者可通过评论和发起者进行沟通确认。
  • 针对提交记录对应的文件改动,进行代码行级评审(评论)。

开始评审

在「合并请求」的「文件改动」页签,代码评审者可以针对代码文件进行逐行评论以完成评审。

鼠标悬停在代码文件中的某一行,界面会显示出 + 号。点击即可针对该代码行进行评论。填写评论之后:

  • 点击「评论」,直接发布该评论。
  • 点击「开始评审」,则发布评论的同时,该合并请求还会显示「评审中」的状态。

评论发布成功之后,评论内容和评审状态(如有)会同步显示在合并请求概览页面的活动日志。

跟踪文件查看进度

当需要评审多个代码文件时,可在评审完一个文件之后点击「已查看」进行跟踪与标记,文件查看进度条会随之自动更新。

文件「已查看」并不影响评审状态,仅用于跟踪文件查阅进度,且只针对当前用户生效。

完成评审

待所有代码文件均已被评审之后,可点击右上角「完成评审」发布评审结果并结束评审。

「完成评审」按钮上的数字代表当前页面所有代码评论的数量。

  • 「评论」:必需填写评论。评论内容为填写内容。
  • 「允许合并」:可不输入评论,直接发布评审结果为:「允许合并」。
  • 「需要改进」:可不输入评论,直接发布评审结果为:「需要改进」。

完成评审之后,评论内容或评审结果会同步显示在合并请求概览页面的活动日志。

若合并请求的目标分支为保护分支,将会显示合并请求的评审状态(只要存在「需要改进」的评审结果,评审状态即为不通过)。若评审结果为「允许合并」,无论目标分支是否是保护分支,该结果还会作为标签显示在分支状态中。

上一篇合并分支
最近更新
感谢反馈有用
感谢反馈没用

在阅读中是否遇到以下问题?

您希望我们如何改进?

工单咨询