前端怎么做 Code Review？

[caisanli]

迫于团队（ 6 人）代码和技术参差不齐

代码风格和代码逻辑没法看

目前项目大部分基于 Vue2 （需要兼容 IE10

也很难上 TS

目前想的方案是：

• ESLint
• 把有问题的代码贴出来

正好公司要实行绩效，我有打分权，是不是可以实行一下手段。

[AyaseEri]

@caisanli ESLint + Prettier 做风格管理，然后 Excel 表格做 Code Review 问题管理，重点还是问题登记与跟踪。

[parrotdance]

我觉得我们团队的做法不错, 可以参考一下:

1. 每个 mr 至少对应一个 issue.
2. 编码之前开一个空 mr, 在其中简述问题原因 /需求拆解 /解决思路, 组员之间 review 并讨论.
3. 编码完成后编辑 mr 内容, 加入测试部分内容(截图或视频), 确认改动是有效的.
4. review 代码改动.

这样做的好处在于别人 review 的时候大概知道你这个改动想干什么, 再去看代码脑子里能够有个总体脉络, 另外换个人来看 mr 里描述的思路和测试结果也容易发现考虑不周的地方.

坏处嘛...可能就是对书面表达能力有一定要求