Skip to content

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

Open
@haizhilin2013

Description

@haizhilin2013
Collaborator

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

Activity

Damon99999

Damon99999 commented on Jun 19, 2019

@Damon99999

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

AricZhu

AricZhu commented on Jul 6, 2019

@AricZhu

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

seho-dev

seho-dev commented on Aug 1, 2019

@seho-dev

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

xcLtw

xcLtw commented on Sep 12, 2019

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

blueRoach commented on Jun 2, 2020

@blueRoach

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

smile-2008

smile-2008 commented on Sep 14, 2020

@smile-2008

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

Gun-yx

Gun-yx commented on Feb 25, 2021

@Gun-yx

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

WangXi01

WangXi01 commented on Jul 13, 2022

@WangXi01

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @smile-2008@haizhilin2013@xcLtw@WangXi01@blueRoach

        Issue actions

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