Code Review

最近更新时间:2019-06-04 10:13:53

创建评审、添加评委

注意:

代码评审设置的【建议评委】和【必要的评委】有添加成员的话,在创建合并请求时会自动创建评审,【建议评委】和【必要的评委】中添加的成员会自动成为该合并请求的评委。【建议评委】可删除,【必要的评委】不可删除。

  1. 新建合并请求,创建成功后的部分截图如下所示。
    img
  2. 单击右侧【新增评审人】,并在下拉框中找到需要参与评审的人员来添加评委。
    img
  3. 添加评委成功之后,页面将会提示 “合并请求被阻止,需要审查 ”,评审改变为锁定状态,即需要通过评委评审才可提交。
    img
  4. 评委收到之后可以在【变更】查看内容。
    img
  5. 可以对有疑问的代码进行评论。
    img
  6. 评委确认没有问题后,可在 【评审意见】进行是否通过的操作。
    img
  7. 当评委评审已经通过后,合并请求将改变为可合并状态。
    img

Code Review 设置

注意:

  • 在项目设置 > 高级设置 > 代码评审中的设置是针对非保护分支的;
  • 在项目设置 > 保护分支的设置是针对保护分支的。且不同的保护分支可按照实际情况进行不同的设置。当合并请求在该分支(作为目标分支)发起时,代码评审会依据配置自动带起。

二者设置方法一样,只是进入的路径不一样。以下以非保护分支的设置为例:
操作步骤: 项目设置 > 高级设置 > 代码评审,单击至如下所示页面。
img

设置详情

特点

  • 特点1:如勾选该选项,在评审未通过时,工蜂和 git 客户端都无法合并此次合并请求,如未勾选该选项,在评审未通过时,可同过本地 git 客户端合并此次合并请求。
  • 特点2:勾选后即使当前审阅状态已获批准,推送新提交也会自动重置状态为未批准 ,不勾选则没有该效应。

评委

  • 建议评委:添加人员后创建合并请求 会自动成为评委且可以被移除。
  • 必要的评委:添加人员后创建合并请求会自动成为评委,但是不能被移除。

分支设置

  • 全评委通过: 是全部评委通过,评审才能通过 ;
  • 单评委通过: 是当某个评委通过,评审即通过 (不设置的情况下默认为单评委通过)。

代码评审状态

  • 需评审: 评审创建后的初始状态,当有成员将该评审的状态改为需要修改拒绝状态后,再次打开这个评审 状态将会重新回到 需评审
  • 需要修改: 每一位评委都可以将需评审状态更改为需要修改,再次打开该评审后状态将会变为 需评审
  • 拒绝: 每一位评委都可以将需评审状态更改为拒绝,再次打开评审后状态将会变为需评审
  • 同意:合并请求的状态变更为通过评审时,即意味着该 合并请求通过评审(注意:关注评审设置是单评委通过 还是全评委通过)。