##无CR,不上线
Code Review的重要性我们这里就不再赘述了,全公司所有上生产的代码,需要强制CR,无CR,不上线。
我们使用的代码仓库系统是GitLab,配合 分支保护 来强制做 Merge Request
设置为不允许往dev和master分支直接push代码,仅允许merge request的方式,设置见下图:
<center>![](https://box.kancloud.cn/c85e3e21fca546269ac505931b98dc8e_374x331.png)
<font color=red>注意下面的 Allowed to merge 和 Allowed to push 的区别,任何人不允许做push。</font>
![](https://box.kancloud.cn/5becd48cae1530f9ad50abefd7b8016b_867x162.png)</center>
下面是一个利用GitLab的Merge Request 做CR的一个引导,供大家学习:
![](https://box.kancloud.cn/65b1abbb9a2544920131e6d2412d15ff_800x1386.png)
##Project Owner 开启 Merge Request Approval 的设置,按照下面的配置设置,强制推行CR
![](https://box.kancloud.cn/a0ec094766ddf4b3a71ace94dd7b7db1_2738x398.png)
![](https://box.kancloud.cn/a13fcb7b3afed597fc993cbfce281545_1906x1400.png)
- 说明
- Python编程规范
- Python风格规范
- Python语言规范
- Java编程规范
- 一、命名约定
- 二、常量定义
- 三、格式约定
- 四、OOP约定
- 五、集合处理
- 六、并发控制
- 七、控制语句
- 八、注释约定
- 九、异常日志
- 十、日志约定
- Android开发规范
- 前端开发规范
- HTML
- JavaScript
- CSS
- MySQL约定
- 一、基本规范
- 二、库表设计规范
- 三、字段设计规范
- 四、索引规范
- 五、SQL设计规范
- 六、业务字段命名规范
- 开发安全约定
- 一、代码安全
- 二、移动开发安全
- 三、服务器安全
- 四、安全意识
- 版本管理
- Git使用规范
- 技术实践及可视化
- 一、Code Review
- 二、单元测试
- 三、自动化测试
- 四、技术债
- 五、CI
- IOS开发规范