研发团队如何借助Gitlab来做代码review

代码review是代码质量保障的手段之一，同时开发成员之间代码review也是一种技术交流的方式，虽然会占用一些时间，但对团队而言，总体是个利大于弊的事情。如何借助现有工具在团队内部形成代码review的流程与规范，是team leader或技术管理者需要考虑的问题。本文分享一种基于Gitlab代码merge流程的code review方法，以供参考与探讨。如有更好的方法，欢迎交流。

1. 设置成员角色

首先需要对你团队的成员分配角色，在Gitlab groups里选择一个group，然后左边菜单栏点击 Members，可在 Members 页面添加或编辑成员角色，如下图所示。

其中角色包含如下几类：

Guest：权限最小，基本查看功能
Reporter：只能查看，不能push
Developer：能push，也能merge不受限制的分支
Master：除了项目的迁移、删除等管理权限没有，其它权限基本都有
Owner：权限最大，包括项目的迁移、删除等管理权限

详细权限参考： https://docs.gitlab.com/ee/user/permissions.html

确定团队中技术水平、经验较好的成员为Master，负责代码的review与分支的合并；其他成员为Developer，提交合并请求，接受review意见；Master之间可以互相review。

2. 配置分支保护

在项目页面左侧菜单栏 Settings -> Repository，进入“Protected Branches”部分配置分支保护，如下图所示。

在这里可以针对每个分支，设置允许什么角色可以merge，允许什么角色可以push，选项包括三个：“Masters”， “Developers + Masters”， “No one”。这里设置成只允许master可以直接push与merge这几个常设分支的代码。（如果更严格一点，可以将“Allowed to push”设置成“No one”）

3. 代码review流程

3.1. 开发（开发者负责）

1、本地切到develop分支，拉取最新代码（相关命令如下，GUI工具操作自行查相关文档）

git branch #查看当前位于哪个分支，前面打星号即为当前分支
git checkout develop   #切换到develop分支
git pull  #拉取最新代码

2、从develop分支切出子分支

git checkout -b feature-1101  #从当前分支切出子分支，命名为"feature-1101"

3、编码、本地自测完之后，提交子分支到远程仓库

git add *  #加入暂存区
git commit -m "commit msg" #提交到本地仓库
git push origin feature-1101 #提交到远程仓库

3.2 发起Merge请求（开发者负责）

1、在项目主页面，依次点击左侧“Merge Requests”（下图1），“New merge request”（下图2），打开新建Merge请求页面3.2 发起Merge请求（开发者负责）

2、在新建Merge请求页面，选择merge的源分支，及目标分支，如下图源分支为“feature-1101”，目标分支为“develop”，点击“Compare branches and continue”按钮进入对比与提交页面

3、在对比与提交页面，可以点击“Changes” tab查看本次修改（这里我为了演示，只是加了两个换行），确认无误，点击“Submit merge request”按钮，提交merge请求

4、提交之后，将结果页面的浏览器地址发到团队即时通讯群（如钉钉），并@相应的同事申请review

3.3 代码Review（code reviewer负责）

1、负责代码Review的同事收到申请后，点击merge请求地址，打开页面，查看“Changes”。这里可通过“Inline”单边查看，也可以通过“Side-by-side”两个版本对比查看

2、review完成后，若无问题，则可点击”Merge”按钮完成merge，同时可删除对应的子分支“feature-1101”；若有问题，则可点击“Close merge request”按钮关闭该merge请求（也可以不关闭复用该merge请求），同时通知开发者进行相应调整，重新提交代码发起merge请求（如果之前没关闭merge请求，则刷新即可看到调整）。

3.4 冲突解决（开发者负责）

1、merge的时候，可能存在代码冲突，这时，开发者可从develop分支重新拉取最新代码进行本地merge，解决冲突后重新提交代码进行review

git pull origin develop #在当前子分支拉取develop分支的最新代码进行本地merge# 解决冲突代码# 提交
git add *
git commit -m "fix merge conflict"
git push origin feature-1101

2、自行解决不了时，寻求协助

4. 借助阿里钉钉机器人来改善体验

前面流程中提醒code reviewer是需要开发者自己来发消息通知的，可不可以把这个流程自动化。我们可以借助Gitlab的webhook与钉钉机器人来实现。

1、在钉钉群右上角点击“…”，打开群设置，群机器人中点击添加机器人，会显示可以添加的机器人类型，如下图所示

2、选择Gitlab，点击添加，输入机器人名称，如“Gitlab”，点击完成即创建了一个Gitlab的钉钉机器人。回到“群机器人”窗口，将能看到刚刚创建的Gitlab机器人，如图

点击齿轮按钮，进入设置页，可看到webhook地址，点击复制，复制该机器人的webhook地址。如图

3、在Gitlab项目主页进入 Settings -> Integrations，将前面复制的webhook地址填入URL中，Trigger 部分选择“Merge request events”（不要勾太多，不然提醒太多就有点骚扰了），然后点击“Add webhook”就完成了。如图

4、当有开发人员提交merge请求时，钉钉机器人将在钉钉群里发出通知，code reviewer点击消息里的链接即可进入页面进行code review， review完成，将分支merge之后，钉钉机器人也会发出消息（所有merge相关的事件都会发出消息）。如图

5. 总结

团队协作，流程、规范很重要，不同的团队可能有不同的适用流程与规范。此文分享了基于Gitlab与阿里钉钉群机器人的代码review流程，希望对团队研发协作有一定参考价值，也欢迎一起探讨、交流。