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

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

17 回复

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

@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。

简历发给我,我给你内推[email protected]🤝

目前团队 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的难题

回到顶部