GitLab9.4的CodeReview应用

安装

官方的安装文档已经非常的清楚了，照着安装相信你行。

关于CR（Codereview）

作为一个技术管理者，是非常渴望做CR，但是苦于会面对各种障碍总是会不了了之。

原因如下：

• 时机很难把握，如果太早没有意思，太晚代码太多看不过来。
• 工期太紧，即便是发现了问题也没有时间修复。
• 。。。

做CR的好处是多多的

• ROI最高，在研发期间发现的BUG，修复的成本是最低的。
• 可以获得快速的反馈，提升人员编码能力。
• 编码经验的传授之道。
• 。。。

GitLab完美支持

CR的思路

• 拆分代码，每次提交Merge Request（MR）作为节点
• 由有经验的人承担Master角色进行Review，并及时反馈改进意见。
• 主要分支进行保护，不允许Push代码，必须发起MR请求。
• 坚持“行行代码要Review，人人代码要Review”

具体实施

• 每个项目的核心分支都是保护的，可以通过 [project settings]配置

  
  
  

  image.png

• 每个开发人员独立创建自己的分支进行开发，命名推荐格式feature-news，开发完成Push到本分支，然后登陆到GitLab页面进行合并请求

  
  
  

  image.png

• master角色进行代码Review

  
  
  

  image.png
• 点击merge完成合并（注意gitlab不支持打回功能，提倡线下快速沟通修改，并在此提交，在此提交在Commits就会出现两个。在此review即可，对于提交者应该是提交MR之后，应该中断工作等待Merge成功并查看效果）
• 整个过程特别简单，关键在于思路的转变。