codeReview规范
review组方向merge权限人员xxx组xxxxxx售前xxxxx支付xxxxxx用户xxxxxx目前codeReview机制开发周期较长或是较重要的需求(由开发者自己把握),由开发者组织线下review,一般在公司约定会议室或是钉钉视频会议,参与评审的人提出修改意见,评审者针对修改意见进行修改开发周期较短且非关键的需求(由开发者自己把握),会由开发者发出线上review(MergeReque
review组
方向 | merge权限 | 人员 |
|---|---|---|
| xxx组 | xxx | xxx |
| 售前 | xx | xxx |
| 支付 | xxx | xxx |
| 用户 | xxx | xxx |
目前codeReview机制
开发周期较长或是较重要的需求(由开发者自己把握),由开发者组织线下review,一般在公司约定会议室或是钉钉视频会议,参与评审的人提出修改意见,评审者针对修改意见进行修改
开发周期较短且非关键的需求(由开发者自己把握),会由开发者发出线上review(Merge Request),参与评审的人提出修改意见,review完后complete此review请求
目前codeReview机制存在问题
没有review
参与Review的人太多了,意见太分散,Review时间拉的很长,发现问题效率低
有时候发现一个CodeReview时间很长,参与者可能会觉得煎熬和浪费时间
有时候不太了解对方评审的东西,没法跟上大家思路,影响效率
有时候走查的代码量太大了,无法做到详细走查,匆匆结束,根本发现不了问题,起不到reivew的作用
CodeReview的目标和原则
CodeReview的目的是提升代码质量,尽早发现潜在缺陷与BUG,降
原创不易,完成人机校验,阅读全文