让团队保持Code Review习惯的三大法宝

原文链接: alili.tech/archive/147…

之前跟大家聊过代码审查,想要在团队中保持团队代码审查习惯,是相当困难的. 我们必须要有合理的流程,工具与制度的支持,才能基本保证我们代码审查效率与质量.

流程支持:Gitflow

之前有介绍Gitflow的工作流.

大致如下:

• 开发者在本地仓库中新建一个专门的分支开发功能。
• 开发者push分支修改到公开的Git仓库中。
• 开发者通过Git发起一个Merge Request。
• 团队的其它成员代码审查，讨论并修改。
• 项目维护者合并功能到官方仓库中并关闭Merge Request。

工具支持

强制使用eslint

强制使用eslint,在代码未提交之前,是用husky等工具做强制eslint. 保证提交之后的代码,必须先过一遍eslint.

规范提交代码的类型

我们自己内部开发了一款简单的命令行工具,可以在我们提交代码的时候,定义本次提交的类型.

方便我们后续在代码审查的时候,更加容易的理解修改的内容.

类型如下

• bug修复
• 新特性
• 样式修复
• 代码重构
• 测试代码
• 代码回滚
• bug修复
• 文档更新
• 临时提交

命令行使用方式

? What do you want to do? 代码提交
? 请选择Git提交类型？ (Use arrow keys)
❯ * fixed    : bug修复
  * feature  : 新特性
  * style    : 样式修复
  * refactor : 代码重构
  * test     : 测试代码
  * revert   : 代码回滚
  * doc      : 文档更新
(Move up and down to reveal more choices)
复制代码

Code Climate

Code Climate是一款代码测试工具,它可以帮助你进行代码冗余检测、质量评估，同时支持多种语言，如PHP, Ruby, JavaScript, CSS, Golang, Python 等。

你可以将他集成到GitLab-CI或者Travis CI中,当代码提交后,会自动给出评估报告,以及修改建议.

gitlab与钉钉

在我现在的公司中,我在gitlab的基础上做了二次开发,当有代码审查任务的时候,可以使用邮件或者钉钉通知到相关人员.

如果以后钉钉DING任务开放api,我们甚至可以使用钉钉来完成我们一切的代码审查任务的管理.

人工的代码审查应该在所有持续集成的工作跑完之后才进行. 这样可以大大的减少我们审查的工作量而且还保证了一定程度的代码质量.

公司的支持

从公司层面上,也应该有相应的措施鼓励代码审查的工作.

• 鼓励员工帮助别人审核代码，甚至可以做到效绩评估中。
• 制定统一的编程规范和代码风格，特别是在那些有争议的地方，可减少很多程序员偏好带来的矛盾。

这是我最近对代码审查的一些所思所想