大厂Code Review 流程

提交cr的流程

• 检查代码风格（可以安装googlestyle或者Alibaba的一些stylecheck工具，也许各开发团队会有自己的风格规范）
• 从mainline中同步代码，注意使用（git pull --rebase）而不是（git pull）
• 测试你的代码，单元测试（UT）覆盖率至少达到90%以上，对于订单相关的组件修改，需要手动完成集成测试（最好通过脚本的方式）
• 一次提交尽可能的只包含一个特性的修改，如果涉及到多个特性的修改，在允许的情况下，分成两个cr提交
• 提交cr命令使用cr --new-review --include package1,package2,后面更新的话，命令为cr -r [cr版本号] --include package1,package2
  推荐的cr描述如下：

 

//[简单可识别的标签: 如FIX,ProjectID][一行简洁的描述你的修改]


### 简短的描述
// 你修改了什么，为什么进行此次修改

// 你在修改的时候，考虑过哪些备选方案，为什么没有选择对应的修改方案

### 修改对应的资源链接
// 需求文档或者bug描述链接

### AB测试名字
// 相关的AB测试配置或者链接（如有）

### 影响范围
// 影响的国家，城市，

### 影响的持续集成环境
// 本次上线影响了哪个持续集成配置

### 测试情况
// 用了哪些测试数据，你是怎么测试的，测试结果可以贴在这

代码check in

cr通过之后，需要进行以下步骤:

• 通过 git rebase -i压缩代码git提交记录
• 通过 git pull --rebase更新代码
• 更新cr cr -r [cr版本号]
• 千万不要通过git push去推送代码，一定要通过cr页面进行merge

整个cr流程就是这样，你学会了吗？，喜欢我就关注我吧！