求推荐一款 code review 协同工具 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
meisei81
V2EX    问与答

求推荐一款 code review 协同工具

  •  
  •   meisei81 2019-01-16 14:27:48 +08:00 2560 次点击
    这是一个创建于 2462 天前的主题,其中的信息可能已经有所发展或是发生改变。

    目前团队在做 code review 的时候,遇到需要修改的地方都是口口相传,所以想问下有没有好一点的 code review 协同工具推荐。

    环境:intellij idea + gitlab

    想达到的效果: 在不搭建额外服务的情况下,review 后提交的修改意见可以让别的成员看到。

    谢谢。

    11 条回复    2019-01-16 17:01:35 +08:00
    ian19znj
        1
    ian19znj  
       2019-01-16 15:59:45 +08:00
    Upsource? Free under 10 users.
    meisei81
        2
    meisei81  
    OP
       2019-01-16 16:07:27 +08:00
    @ian19znj upsource 似乎需要搭建额外的服务的吧?
    ian19znj
        3
    ian19znj  
       2019-01-16 16:09:24 +08:00
    @meisei81 #2 不晓得哇,我没用过,就是突然想到,没帮上忙。
    unifier
        4
    unifier  
       2019-01-16 16:15:30 +08:00
    gitlab 本身的 merge request 系统里不就可以直接对代码进行 comment 吗,也可以对 MR 本身进行 comment 啊
    feiyuanqiu
        5
    feiyuanqiu  
       2019-01-16 16:17:11 +08:00
    gitlab 自带了 review 功能吧,提交 Merge Request 之后可以直接在变更代码上添加 review comment
    hei1000
        6
    hei1000  
       2019-01-16 16:26:58 +08:00
    review 我们公司都是 gerrit
    meisei81
        7
    meisei81  
    OP
       2019-01-16 16:34:11 +08:00
    @ian19znj 嗯嗯。感谢回复~
    meisei81
        8
    meisei81  
    OP
       2019-01-16 16:34:53 +08:00
    @hei1000 gerrit 需要搭建额外服务么
    meisei81
        9
    meisei81  
    OP
       2019-01-16 16:35:30 +08:00
    @feiyuanqiu 有没有在本地直接 review 的
    hei1000
        10
    hei1000  
       2019-01-16 16:36:52 +08:00
    @meisei81 #8 具体不清楚,你可以查一下,我觉得应该需要吧,不过哪个公司没服务器呢
    Mithril
        11
    Mithril  
       2019-01-16 17:01:35 +08:00
    @meisei81 只是本地做的话,你可以直接用 TODO 注释。然后把 Intellij 里 TODO 窗口输出的内容复制出来发给同事。
    不过都上了 Gitlab,为什么不直接走 PR ?
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     2916 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 29ms UTC 14:32 PVG 22:32 LAX 07:32 JFK 10:32
    Do have faith in what you're doing.
    ubao snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86