代码评审总是走过场?把清单变成团队资产
几乎每个团队都有代码评审,但真正从中获得质量的团队并不多。常见情形是:提交人自己先看一遍觉得没问题,评审者扫一眼 diff 留下一句“没问题”,合并。评审...
代码评审不是走过场:小步提交让Review真正有效
拉了一个三百行的合并请求(PR),挂了两天没人理;好不容易有人回复,只有一句“LGTM”;合并上线后出了事故,大家才发现问题就明晃晃躺在那一大段代码里。这...
代码评审流于形式?抓住这四类真问题
很多团队的Code Review是这样进行的:提交者把代码往群里一甩,第二天没人看,或者只有一句”LGTM”;好不容易有人评论了,争论的却是变量命名和...
Code Review流于形式?把改动拆小才能审出真问题
周一早上,你收到一条消息:“帮我 review 下这个 PR。”打开一看,600 行改动横跨五个文件,功能逻辑里还混着重构和格式调整。你硬着头皮看了半小时,提了几...