小忆微软的codereview步骤

yueq 2013-02-06 10:30:27 +08:00 8821 次点击

这是一个创建于 4639 天前的主题，其中的信息可能已经有所发展或是发生改变。

无意中看到 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的工具，把这个过程略自动化了，不知道现在有没有改善。

邮件

transfer

dpk

6 条回复 1970-01-01 08:00:00 +08:00

jedia9264

2013-02-06 10:52:15 +08:00

咋个这么原始呢？堂堂微软啊。他自己不是有 TFS，都不用啊？

AlexYang

2013-02-06 15:43:29 +08:00

VS2008里新开发得Team Foundation Server不知道有没有code review的功能。
现在开源社区的CI工具链都已经很自动化了。

missdeer

2013-02-06 17:37:54 +08:00

ms不同的项目组用不同的vcs。现在那个code flow集成了一些工具，已经非常强大了，可以多轮迭代review，可以comment。。。而且也不是发邮件发起code review了，在bugger里改status就会把diff整出来搞。。。

200

2013-02-07 11:53:04 +08:00 via iPad

去年实习的时候Team用的是codeflow，现在codeflow很好用，可以和sd集成，自动生成dpk

yueq

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

2014-03-1 21:30:47 +08:00

感觉就是上个世纪的工具，又慢又难用。
哎，日薄西山了~