V2EX 首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  职场话题

你们都是怎 code review 的?

  •  1
     
  •   jadetang · 127 天前 · 1830 次点击
    这是一个创建于 127 天前的主题,其中的信息可能已经有所发展或是发生改变。
    各位 V 友 来说说,你是在工作中是怎么 code review 的吧?
    第 1 条附言  ·  127 天前
    我们有一个用 一个 git 的插件,提交的时候,会自动在系统上发起 review。review 的人 accept 了以后,才会把你的代码合到主分支里面去。
    12 回复  |  直到 2017-05-22 20:37:11 +08:00
        1
    caixiexin   127 天前 via Android
    上家公司一开始有,后来换领导就没了。
    就是各自上台把自己最近做的代码讲一下,特别是核心模块。下面的人有意见会提出来。
    review 还是可以学到东西的,要不然天天闷头写代码,不交流一下很多技巧都不知道。
        2
    wph95   127 天前
    phabricator
        3
    yidinghe   127 天前 via Android
    频繁出问题才考虑代码复审。
        4
    hiluluke   127 天前
    提 pull request,没有人 view 是不能合并到 master 的。
        5
    kaneg   127 天前 via iPhone
    gerrit
        6
    luxinxin   127 天前
    每个代码目录下有一个 owner (或者父目录的 owner ),owner 通过才能提交。
        7
    chiu   127 天前
    有新的大功能的话,编码后会组织相关人员,组长等人开 review 会,通过后才集成
        8
    SourceMan   127 天前 via iPhone
    pr 工作流
        9
    skadi   126 天前 via Android
    pr 呗,然后审查。我有个学长给我讲他公司提交的代码首先要通过脚本的风格审查,比如命名或者其他规范,然后自动跑测试。通过了才给上面人审查,最后才是合并。感觉我司鶸到就只有一步。
        10
    qianguozheng   126 天前
    @skadi 我觉得这样挺好
        11
    jadetang   126 天前
    @skadi 什么公司?为什么我经历过的公司都没有这种。
        12
    zergmk2   126 天前 via Android
    bitbucket
    DigitalOcean
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   鸣谢   ·   1004 人在线   最高记录 3541   ·  
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.7.5 · 56ms · UTC 17:05 · PVG 01:05 · LAX 10:05 · JFK 13:05
    ♥ Do have faith in what you're doing.
    沪ICP备16043287号-1