研发团队如何借助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. 开发(开发者负责)

本地切到develop分支, 拉取最新代码(相关命令如下,GUI工具操作自行查相关文档)

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

内容版权声明:除非注明,否则皆为本站原创文章。

转载注明出处:https://www.heiqu.com/wpxpfd.html