提到
code review
,相信大家对此并不陌生,各大厂也时不时的会有各种文档出来,这是一个很深的话题,也是一门玄学,接下来的文档里以我有限的认知来聊聊我所理解的code review
。
code review 流程图片来源[1]
曾经流传一个段子,一个新手提交了10行代码,然后收到了500个的issue
, 有一次提交了500行代码,收到了0个issue
,虽然是一个段子,从中我们不难看出一些问题:
代码风格的统一在每个公司落地实践的方式各不相同,有的公司是通过IDE的各种插件来实现代码风格的统一(不过这种方式也不好统一,毕竟有的人买了idea
, 有的人喜欢用vscode
), 有的公司是通过pre-commit
或post-commit
方式来实现代码风格的统一。
如果你作为code review
的人,看到提pr
的人的代码不知表达什么意思的变量或函数命名,或者一屏幕看不到头的一行代码,你心里是否会mmp, 还有继续看下去的心情么。
Google Style Guides[2]
Commit Message
信息的标准化也是一个重头戏,作为code review
的人,看到一句修复bug
的commit
信息,肯定是有砸电脑的冲动,所以统一规范化的信息提交是一个团队要遵守的规范之一,要不然的话,那就呵呵了。可以参考下下面的文档,看看别人家的Commit Message
规范是如何做的。
优雅的提交你的 Git Commit Message[3]
我个人的感受是看团队规模大小吧,小规模团队的情况下,放pre-commit
下做code style
的风格统一的动作足够满足需求,大些的团队多半会把这些工作放到CI
里去做,大家可以看下Github Action
,自定义组合感受下,常规的语法检测,代码格式化操作等等工作。
如果一个团队中没有达成共识,或者无法取得认可的场景下,强推code review
的结果可能不是很理想,另外一个方式就是自上而下的强推,需要一个漫长的适应期。
这个涉及到前面我们提到的分支模型的选择,在分支模型落地的基础上(不允许用于直接提交代码,只能提PR
),用户提交PR
, 至少需要两个人进行Approve
(投票,bitbucket
中称作点赞)才允许进行Merge
的动作。
Code Review
平台)基于python语言的rietveld[4]
reviewboard[5]
phabricator[6]
gitlab code review[7]
Gerrit[8]
The 8 Best Tools For Code Review In 2020[9]
•AccessLint•Imgbot•LGTM•DeepSource•Codelingo•DeepScan•CodeScene•FeaturePeek
Google Engineering Practices Documentation[10]
Code review checklist[11]
Code Review Review is the Manager's Job[12]
How to give great code review feedback[13]
How one code review rule turned my team into a dream team[14]
code-review-tools[15]
code review
是一门玄学,就像人跑步一样,刚开始的时候你会觉得,真特么累,这不是自己给自己找事么,过了适应期,你会发现,真香~
时间久了你会发现,市面上开源的这些工具不足以满足需求,二开是不可避免的趋势。
虽然大家都在口口声声的说着code review
, 但是真正用起来的团队很少很少。说到这个不得不提TDD
, 自己领会吧~
[1]
code review 流程图片来源: https://www.freecodecamp.org/news/how-code-reviews-work-at-microsoft-4ebdea0cd0c0/
[2]
Google Style Guides: http://google.github.io/styleguide/
[3]
优雅的提交你的 Git Commit Message: https://juejin.im/post/5afc5242f265da0b7f44bee4
[4]
基于python语言的rietveld: https://github.com/rietveld-codereview/rietveld
[5]
reviewboard: https://demo.reviewboard.org/r/
[6]
phabricator: https://www.phacility.com/phabricator/
[7]
gitlab code review: https://docs.gitlab.com/ee/user/analytics/code_review_analytics.html
[8]
Gerrit: https://www.gerritcodereview.com/
[9]
The 8 Best Tools For Code Review In 2020: https://featurepeek.com/blog/the-8-best-tools-for-code-review-in-2020/
[10]
Google Engineering Practices Documentation: https://google.github.io/eng-practices/
[11]
Code review checklist: https://dev.to/uday_rayala/code-review-checklist-20c6
[12]
Code Review Review is the Manager's Job: https://hecate.co/blog/code-review-review-is-the-managers-job
[13]
How to give great code review feedback: https://www.michaelagreiler.com/great-code-review-feedback/
[14]
How one code review rule turned my team into a dream team: https://medium.com/inside-league/how-one-code-review-rule-turned-my-team-into-a-dream-team-fdb172799d11
[15]
code-review-tools: https://kinsta.com/blog/code-review-tools/