基于 GitLab 的代码审查

Code Review 流程

根据 Git Flow 工作流，开发人员在 feature 分支进行开发，开发完成之后 Merge 到 develop 分支进行测试。

那么最适合做 Code Review 就是 featrue 分支合并到 develop 分支的环节

注：为了保证必须以 Merge 的方式变更 develop 分支、release 分支、以及 master 分支，我们对 Push 以及 Merge 权限进行限制

项目仓库 -> 设置 -> 仓库设置 -> Protected Branches

Protected Branches 配置

这里，我们限制分支，所有的开发人员对 develop 分支、release 分支、以及 master 分支均无 Push 权限，只能以 Merge 方式合并到对应分支，而且只有 Maintainers（Masters） 组的用户有 Merge 权限。

Code View 示例

准备工作

变更 feature 分支 分支 README.md 文件，为 Merge Request 提供基础

随意更改一行内容，然后 Commit Changes 即可

创建 Merge Request

左侧菜单 -> Merge Request -> 点击 New Merge Request

Source branch 选择：feature/test

Target branch 选择：develop

然后：Compare branches and continue

操作项/填写项说明：

操作项/填写项	说明
Title	标题，没有特殊要求保持默认即可
Description	描述，需要将变更的需求描述清楚，最好附件 Code Review 要点
Assignee	分配到的人，被分配到的人将会收到邮件通知，跟 Merge 权限没有必然关系，仍然是项目的 Maintainers（Masters）角色拥有 Merge 权限
Milestone	里程碑，如果没有可不选
Label	标签，如果没有可不选
Merge options	合并选项，可以勾选当合并请求被接受时，删除源分支或压缩请求

Merge Request 操作

Merge Request 创建之后就会转到该页面，被分配到的人（Assignee）会收到邮件提醒，如果需要多个人进行 Code Review，只要将该页面的链接发给其他项目成员即可。项目成员可以查看变更并评论，只不过按照之前的配置，只有 Maintainers（Masters）角色的成员才有 Merge 的权限。

在 变更（Changes） 选项卡中，我们可以看到所有的变更。将光标移动到行号处会出现评论按钮，我们可以点击评论按钮发起评论，这个评论是对项目成员可见的，大家可在讨论区进行讨论。最终讨论发起者有权将讨论标记为已解决 resolved

当所有的问题已解决之后（如果选择了审批人也需要审批通过），Maintainers（Masters）成员点击 合并（Merge） 完成合并即可。

Merge 完成之后，可以选择 删除源分支（Remove Source Branch） 等操作。

develop 分支合并到 release 分支，以及 release 分支合并到 master 是不需要经过 Code Review 的，直接 Merge 即可。这里就省略了…