团队成员之间的 code review 大家都用什么工具?

2012-06-09 01:08:59 +08:00
 Elfe
以前在windows平台上的TFS,做CodeReview非常简单:
开发者在自己机器上改好代码后生成一个shelveset,这个shelveset直接就保存在服务器上了
邮件发送给reviewer告知shelveset名称
reviewer通过tfsodd等工具,打开shelveset,查看代码变化,提意见。reviewer本机并不需要有服务器端代码
review结束后(也许几个来回),开发者check-in shelveset。结束。
也有时候,有些shelveset并不check-in,但一样能留在服务器上以备今后查看。

现在的小公司里,也不知道这群毕业了直接就在这里的小朋友,是没人教不会用呢,还是真的Mac下没有好的code review工具,目前他们要么是直接跑到对方座位上看(于是匆匆而过看得很不仔细),要么就是看天书般的svn diff得到的文件(我觉得这个实在需要超能力才能读)。

我粗粗搜了一下,没找到什么方便的工具,只有若干零星功能的,要合并在一起用才好。
所以我在想,是不是该写个脚本或app。
我大致想要一下几个功能。大家帮我看看,有没有能满足我要求的推荐一下。要没有我就只能自己动手了。
0,每个人在机器上都有一个专门用来做code review 的 workspace。配制好CRWorkSpace的路径参数给下面的工具用
1,开发者在写完程序,敲 cr -create <shelvesetName> <message>
于是
1.1 程序检查开发者当前代码是否update过,没有,提醒并帮助svn update,如有必要弹出FileMerge
1.2 svn diff > shelvesetName.patch, 上传patch到公司的share folder
2, cr -email <shelvesetName> <reviewer's email>发邮件给指定email。邮件中包含一条用以apply patch的 cr -review <shelvesetName> 命令
3,收到邮件的同学copy & paste & run 这条命令
3.1 将程序检查code review workspace的代码版本,sync到和patch一致
3.2 apply patch
3.3 弹出 SmartSVN(免费版)以查看代码自之前一个版本起的变动(可以吗?若不行, reviewer 本机还得准备两个work space,用文件比较工具来查看)
4,开发者执行 cr -commit <shelvesetName>提交代码

基本上就这样,还算蛮简单的。
和原来用TFS相比,就差一个reviewer本机必须有一个用来做code review的专门workspace。这个在以后有多个brunch的时候会遇到麻烦(我原先的项目,最多的时候有十多个brunch同时在做平行开发)

各位都在用什么工具,有可以让我偷懒不自己写的么?

谢啦!
35741 次点击
所在节点    iDev
23 条回复
qiukun
2013-02-07 23:41:07 +08:00
faywong8888
2015-01-23 16:51:02 +08:00
@kaichen gitlab适合项目维护者review贡献者也就是一对一的review。不知道gitlab有没有多对一的review方案?
faywong8888
2015-01-23 16:51:50 +08:00
@clino gerrit用过两年,专注于做code review,非常好用且高效。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/38858

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX