无意中看到 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的工具,把这个过程略自动化了,不知道现在有没有改善。
回想两年前在微软某组工作时,发送一个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的工具,把这个过程略自动化了,不知道现在有没有改善。