无意中看到
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的工具,把这个过程略自动化了,不知道现在有没有改善。
这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。
https://www.v2ex.com/t/59935
V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。
V2EX is a community of developers, designers and creative people.