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进行代码合并
推荐:
Git中Fork使用
参考:
/p/5d764b52ea88
/p/257857632
/qq_42183414/article/details/123145014
/lazio10000/p/
/a/1190000016155779