Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[软技能] 第16天 你在的公司有没有做代码审查(CodeReview)?如果有是怎么做的?如果没有你觉得应该怎么做才更好? #53

Open
haizhilin2013 opened this issue May 1, 2019 · 8 comments
Labels
软技能 软技能

Comments

@haizhilin2013
Copy link
Collaborator

第16天 你在的公司有没有做代码审查(CodeReview)?如果有是怎么做的?如果没有你觉得应该怎么做才更好?

@haizhilin2013 haizhilin2013 added the 软技能 软技能 label May 1, 2019
@Damon99999
Copy link

Damon99999 commented Jun 19, 2019

不知道您说的是哪一种codereview
1:提交代码会把代码link发群里,全员都可以进行codereview(大佬一定会过一遍)有不合理代码直接提comment改好了再合并
2:每周一次的codereview全员参加,指定两位至三位小伙伴将本周开发的内容拿出来全部过一遍,全员现场提问现场解答,时间大概1~3h

@AricZhu
Copy link

AricZhu commented Jul 6, 2019

公司采用gerrit服务器进行代码的统一托管,一般组内开发人员把代码git push上去以后,需要有权限的人进行code review,并且通过,然后才能合代码进入生产服务器

@seho-dev
Copy link

seho-dev commented Aug 1, 2019

小公司没有,但是越小公司我觉得越要搞这样的,因为能很提高代码水平,全员监督你的代码,是非常好的做法

@xcLtw
Copy link

xcLtw commented Sep 12, 2019

  1. 有独立的代码审查部门,定期发送邮件给相关人员,里面有本部门全部项目的代码质量统计,在代码过差时依次向上级发通知
  2. 依据每个组内风格,有的组在每次合并生产环境都会review
  3. 总的来说代码审查是好事,但如果出现咸鱼池塘以及产品流程不规范导致迭代需求过多而不合理,会造成很多困扰,自身也可能流于形式,一定要结合实际情况来看

@blueRoach
Copy link

目前没有,不过确实有必要,以后会尝试做的
具体的,比如每周五下午抽几个小时把自己本周的代码过一遍。
1.肯定会发现很多可以优化的地方
2.也可以复习下,在写代码的时候查资料的一些没有掌握的知识

@MrZ2019
Copy link

MrZ2019 commented Sep 14, 2020

不知道您说的是哪一种codereview
1:提交代码会把代码link发群里,全员都可以进行codereview(大佬一定会过一遍)有不合理代码直接提comment改好了再合并
2:每周一次的codereview全员参加,指定两位至三位小伙伴将本周开发的内容拿出来全部过一遍,全员现场提问现场解答,时间大概1~3h

@Gun-yx
Copy link

Gun-yx commented Feb 25, 2021

1.每天要把代码放进项目组里,组长会来检查,有问题的话会直接进行沟通

@WangXi01
Copy link

以任务为导向,一般不做review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
软技能 软技能
Projects
None yet
Development

No branches or pull requests

9 participants