code review的目的是提高代码质量，减少开发bug，俗话说，三人行必有我师，众人拾柴火焰高。gitlab提供了code review机制，对基于gitlab的code review，直接以具体例子的形式做个实践总结。
gitlab提供了两种代码merge机制：
- 在本地将源分支(Source branch)代码合并到目标分支（Target branch），然后Push到目标分支（Target branch）。
- 将源分支(Source branch)Push到远端，然后在GitLab指定目标分支（Target branch）发起Merge Request，对目标分支（Target branch）拥有merge权限的用户执行Merge操作，完成合并。
这两种方式仅有第2种适合code review，所以我们要做的事情是设置权限，拒绝本地merge后push到远端的操作。在第2种方式中 发起merge request后，由有merge权限用户做code review，通过后执行merge操作。
分支设置
创建项目分支
经典的 git flow 图中，有 Master、Hotfix、Release、Develop、Feature 等分支。每个分支的作用各不相同。各个分支的主要功能如下：
例中假设流程开发如下：
-
每次需要新feature需要开发时，则从develop上拉取feature分支，因此develop分支是feature的父分支
-
master有更新及时合并到develop-1，develop，以及release。
-
develop-1开发完成后合并到develop，部署测试环境。
-
develop环境测试通过后，合并develop-1代码到release环境，做预发布测试。
-
release环境测试通过后，将develop-1代码合并到master，上线。
设置分支保护
为了保证必须以Merge的方式变更develop分支、release分支、以及master分支，我们需要对Push以及Merge权限进行限制。我们限制分支，所有的开发人员对develop分支、release分支、以及master分支均无Push权限，只能以Merge方式合并到对应分支，而且只有Maintainers（Masters）组的用户有Merge权限。如下图：
设置仓库用户权限
图下图，zhangsan作为项目的owner，lisi作为项目的developer。按照上面的分支保护规则，lisi是不能够从develop分支merge到其它分支的，zhangsan是可以执行merge的。
合并请求
创建
lisi打算将develop分支代码合并到test分支。lisi在本地将代码commit、push到develop分支，然后登录gitlab在对应的项目下提交merge requests。如下图：
如下图，准备创建merge requests。source branch选择develop，表示需要merge的分支；target branch选择test，表示需要合并的目标分支。核心逻辑表示创建一个从develop分支merge到test分支的请求。
如下图，填写一些merge request的表单参数。titile表示本次merge request的简短标题；description表示本次merge request的详细说明，在这里介绍一下本次merge request做了哪些改动；assignee选择自己（还没研究出具体干啥用的）；reviewer表示需要通知谁来做代码检查工作；milestone表示里程碑选项，如果有特殊要求的话可以选一个，通常都是选择no milestone；labels表示项目标签，如果有特殊要求的话可以选一个，通常都是选择no label。最后点击create merge request。
当lisi将merge request创建完成以后可以在list中看到列表，如下图：
lisi查看merge request详情，如果有问题或者需要重新修改的可以点击最后的close merge request，如下图：
到此，lisi就完成了merge request的创建，接下来就等待具有merge权限的zhangsan来执行merge操作。
合并
合并需要使用zhangsan进行操作，登录进来以后在列表中以及右上角都会给出merge request的提示。如下图所示：
点击进入merge request详情有两块，如下图：
- 关于change和commit
- 在commits中可以看到此次merge request的commit情况
- 在chanage中可以看到此次merge request中所有的文件变动情况
- 关于approve
- 被指定reviewer的用户在reviewer通过以后可以点击approve审核通过
- 关于merge
- 具有merge权限的用户可以操作merge进行代码合并
参考：
www.jianshu.com/p/5d764b52e...
zhuanlan.zhihu.com/p/257857632