code review机制
发布于 10 天前 作者 Rabbitzzc 1138 次浏览 来自 问答

请问一下大家有好的code review机制么? 平常团队review的时候总是会出现偏差,比如突然讲到了业务细节,感觉效率好低,求大佬们分享和建议, 评论也会给其他需要review的coder建议🤗

22 回复

请问代码托管在什么上面呢?

@sunNode 内部仓库

@sunNode 算作gitlab吧

gitlab 里面不是可以提 PR 的啊,不过不可以设置多个 Reviewer,指给指定 Reviewer,在提交 PR 的模板里面关联 case 编号,介绍这个 PR,还有就是 code commit message规范就可以了。

同求code review 最佳实践~

我现在的团队没有 review, 但是我想说下理想中的场景:

  1. review 应该是异步的, 不需要强制把大家叫到一起, 盯着大屏幕 review
  2. review 不能光看逻辑, 代码风格, 实现是否优雅也应该是重点 review 范围
  3. 明确一点, review 不是给别人挑错, 而是一场技术交流, 差的指出来使人进步, 好的赞美一下向人学习
  4. 不一定非要熟悉该业务、模块的人 review, 每个人各有所长, 别人站在不同的视角可能有意想不到的收获 个人想法, 欢迎探讨指点 如果贵公司是这样的, 并且有前端 / node岗位的话, 欢迎联系我

@sunNode 这个应该算PR。Code Review会议还是要有的,这样大家可以一起学习学习,一起讨论。

@yinxin630 https://juejin.im/post/5d550f776fb9a06ae439ee81。这是我今天写的,也是问了很多人&查了很多资料,可以看一下。code review会议是需要有的,大家一起讨论会更加注重代码的最佳实践。还有一点就是code smell,一起讨论的时候有经验的同事会很快发现代码中的code smell。

简历发给我,我给你内推1798448128@qq.com🤝

目前团队 Code Review 已经跑了一年多了,效果不错,可能不算最佳实践,但是目前也没有不爽的体验。基于企业微信 & GitLab,截取部分内容供参考~ image.pngimage.pngimage.png

@yinxin630 目前在美团住宿,团队在内推呢,不知道你感兴趣不

@guchongxi 这个应该就是PR吧

@Rabbitzzc 感谢,目前还没有这方面考虑哈

@Rabbitzzc 是的,GitHub - PR; GitLab - MR 一个东西

当然目前我们是约定优先,也可以使用 GitHub 的 Require pull request reviews before merging 或者 GitLab 的 Merge request approvals 来限制必须经过指定成员 Review 后才能合并分支

@Rabbitzzc 初期可以搞一些集体会议, 团队养成习惯后尽量减少, 不然总有种逼人的感觉, 达不到什么好的效果. 更主要还是费时间…

@yinxin630 所以review目标很重要,把握方向是review的难题

@Rabbitzzc 嗯嗯,我同意你的说法。但是实际开发过程中,不太可能有这样的会议,第一点是项目大了之后每个模块细致的方面不是每个成员都清楚,一般是小组 Leader 比较清晰;第二点这种 Review 会议比较耗时,如果改动点较大的话无法从 Review 会议中找出问题。所以我的建议还是从源头把控,从功能需求开始做的时候需要有 Design 会议,同时可以请组内小伙伴来听,最后从了解需求的小伙伴里面挑选 Reviewer。这样 Reviewer 可以根据的 PR 里面的描述,以及对需求的了解作出判断。我们之前的做法一般会将 PR Code 拉取到本地,结合未改动的 Code 给出建议,这样 Review Code 不会成为 Reviewer 的负担,也可以很快的给出回应。

@sunNode 感谢建议

嘿嘿,学习了

回到顶部