如何使用Gitlab设置代码审查?

如何使用Gitlab设置代码审查? 我在Gitlab网站上看到它被列为一个function,但我似乎无法find如何设置一个(关于这个问题,任何链接到Gitlab用户手册将不胜感激)。

我的一些search表明,“合并请求”是要走的路,但我发现它们是有限制的。 已发布的合并请求显示一个分支与另一个分支之间的所有提交。 我似乎只能查看为每个单独的提交生成的差异。 例如,假设我有一个我想查看的文件。 这是一个新文件,但我已经提交了超过10个提交分支上的更改。 如果我从集成发出该开发分支的合并请求,我看到10个提交每个显示对文件进行的增量更改…我想审查整个事情。 它是新的!

我在这里叫错了树吗? 有没有一个实际的代码审查工具,我可以在GitLab中使用,或合并请求的方式去,如果他们是我使用它们不正确? 在这里build立一个合适的代码审查的最佳方式是什么?

注意:由于GitLab 6.4, 并行差异视图可用:请参阅“ 请求5308 ”。

(2013年7月) 尽pipe只有在文件级别,但是没有可能对每行进行评论。
Daniel Sokolowski 在评论中提到现在支持每行注释 (09/2014):

您的团队成员可以对合并请求进行一般性评论,也可以在特定的线路上留言。

这仍然可以帮助代码审查活动。

assets/4224518/1558702/e0fe633a-4fa3-11e3-9388-3f3e445cb6d4.png

我已经在Gitlab做了两个多月的代码审查,几乎没有任何摩擦。 每次开发者推送新的提交时,我都设置了rss2email来发送电子邮件通知。 然后我使用Gitlab的评论function提交一些关于推送代码的评论。

不幸的是,Gitlab不允许对文件本身进行评论,只是在提交(就像Github,我猜)。 每当我发现自己处于一种情况时,我需要评论之前提交中遗漏的某些内容,我使用责怪工具来查找引入/更改要评论的代码段的提交。

这是远远不够完美,但迄今为止工作良好。

您可以在合并请求中查看其他存储库或当前存储库中提交的代码。
例如http://demo.gitlab.com/diaspora/diaspora/commits/master

然后,您可以添加注释到提交的文件更改(button回复 )或整个提交

示例http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

由此产生的沟通是代码审查 。 不过,我个人build议尽可能在一台个人电脑上进行代码审查和面对面沟通,并使用工具logging结果,或者在需要更多正规性时使用。

对于有很多提交的文件修改,例如http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md使用;责备来理解谁做了什么。 但是在这个视图中,没有select来交stream和添加评论。 我会build议在这种情况下添加更改作为注释。

是。 合并请求是如何完成同行评审。

应该有一个“差异”选项卡,显示所有提交的变化(这里提到:http: //youtu.be/DyAX8ws5OIc?t=3m2s )。

该video也很好地解释了如何将其用于同行评议。

Interesting Posts