V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
yueq
V2EX  ›  程序员

小忆微软的codereview步骤

  •  
  •   yueq · 2013-02-06 10:30:27 +08:00 · 8105 次点击
    这是一个创建于 4068 天前的主题,其中的信息可能已经有所发展或是发生改变。
    无意中看到 http://www.v2ex.com/t/38858 这个thread,讨论如何进行code review。


    回想两年前在微软某组工作时,发送一个code review需要经过以下几步,
    (source depot是微软自己包裹的perforce)
    1. 通过source depot找到changelist的编号
    2. 通过DPK命令生成一个.dpk文件 (在本机)
    3. 将.dpk文件拷贝到\\transfer\transfer\your_name
    4. 手动生成一个full path, e.g. \\transfer\transfer\your_name\changelist_num.dpk
    5. 打开outlook,发一封邮件给reviewers,将full path粘贴到邮件正文。

    除了DPK, 还有诸如bbpack, jjpack, 等等xxpack,yypack各种各样的pack云云

    对于code reviewers来说,没有统一的工具来打开dpk. 默认的是windiff. 可用性很低。我个人使用一个叫odd的内部工具。

    添加comment就更搞笑了。首先新建一封邮件,在odd中,对于某个要发表评论的change点右键,选择“copy”,再在邮件中选择“paste”,这时change会被拷贝到邮件中,并添加了一个小文本框,reviewer在小文本框中填写comments。如果change比较多,那么这个复制粘贴的过程会狠冗长。最后可以在邮件中说些overall这个CR如何的话。邮件中全是change,毫无code context。

    后来内部做了一个叫做code flow的工具,把这个过程略自动化了,不知道现在有没有改善。
    6 条回复    1970-01-01 08:00:00 +08:00
    jedia9264
        1
    jedia9264  
       2013-02-06 10:52:15 +08:00
    咋个这么原始呢?堂堂微软啊。他自己不是有 TFS,都不用啊?
    AlexYang
        2
    AlexYang  
       2013-02-06 15:43:29 +08:00
    VS2008里新开发得Team Foundation Server不知道有没有code review的功能。
    现在开源社区的CI工具链都已经很自动化了。
    missdeer
        3
    missdeer  
       2013-02-06 17:37:54 +08:00
    ms不同的项目组用不同的vcs。现在那个code flow集成了一些工具,已经非常强大了,可以多轮迭代review,可以comment。。。而且也不是发邮件发起code review了,在bugger里改status就会把diff整出来搞。。。
    200
        4
    200  
       2013-02-07 11:53:04 +08:00 via iPad
    去年实习的时候Team用的是codeflow,现在codeflow很好用,可以和sd集成,自动生成dpk
    yueq
        5
    yueq  
    OP
       2013-02-07 12:03:12 +08:00
    @jedia9264 TFS不是所有组都用的。据我所知SD在微软内部还是主流。

    @missdeer @200

    haha, 我软这两年进步蛮大的么 文中描述的情况一直到2011年还存在。 codeflow才搞了没多久吧

    不过codeflow还是微软的一贯风格——software。 但如Google的CR工具都是web-based,每个人都可以看任何人的change和comments.
    weibest
        6
    weibest  
       2014-03-16 21:30:47 +08:00
    感觉就是上个世纪的工具,又慢又难用。
    哎,日薄西山了~
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5906 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 47ms · UTC 02:10 · PVG 10:10 · LAX 19:10 · JFK 22:10
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.