前言
现在很多大厂(google、facebook)已经很长时间用pr来做code review,可能你会说那是体量大、人数多的公司才会这么做、其实不然,其实小团队灵活度更高,对于身边的同事代码更为熟悉、更容易提出针对性意见。而大公司大项目量大、人多、风格不统一,所以codereview也会更难
国内公司追求敏捷开发 、节奏快,是放弃了pull request机制
国外公司基本节奏比较慢、他们有足够的时间对每个童鞋的pr经行评审、提意见、最后merge request
痛点
那么多人团队在开发项目中,如何做到以下几点呢?
- 代码规范、代码优雅
- 逻辑清晰、可复用性强、无重复功能代码
- 做好了自己的功能,同时也非常了解其他业务模块
- 高效阅读别人代码的能力
- 高约束性、避免团队成员因偷懒写一些 可读性差、不健壮的代码
- 风格统一,看起来像一个人写的代码
其实团队内实行pull requset 对代码流程化管理、是可以长期有效的提升整个团队内的氛围、编码、阅读、风险管控等诸多方面。
步骤
1、大版本方式提交PR 平均2周一次code review 、持续数天并计算在排期内、一个重要版本一次
2、小版本方式提交PR、日常review,需要维护一张所有童鞋提交pr记录表、每天可以不定时间随机抽取某位同学pr review、并在表上维护状态
总结
团队中持续有效运用pr 可以养成成员编码习惯、阅读能力、对业务理解能力,持续高质量输出。
好的代码管理流程是打造一支战斗能力强、风险输出少的Team必备方式。
~完,我是Steven,偷偷摸摸写完了,继续搬砖。