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

Question

如何使用Gitlab设置代码审查？我认为它在Gitlab网站上列为一个功能,但我似乎无法找到如何设置一个的说明(就此而言,任何Gitlab用户手册的链接都将非常受欢迎).

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

我在这里吠叫错了吗？是否有一个我可以在GitLab中使用的实际代码审查工具,或者是合并请求的方式,如果他们是我使用它们不正确？在这里设置正确的代码审查的最佳方法是什么？

Answer 1

注意:由于GitLab 6.4,并排差异视图可用:请参阅" 拉取请求5308 ".

(2013年7月)但是,只有在文件级别,才有可能对每一行进行评论.
Daniel Sokolowski 在评论中提到现在支持Per line评论(09/2014):

您的团队成员可以在一般情况下或在具有行注释的特定行上对合并请求发表评论.

这仍然有助于代码审查活动.

Answer 2

我已经在Gitlab进行了两个多月的代码审查,几乎没有任何摩擦.每次开发人员推送新提交时,我都会设置rss2email来发送电子邮件通知.然后我使用Gitlab的注释功能进行提交,以对推送的代码做出一些评论.

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

它远非完美,但到目前为止它仍然运作良好.

Answer 3

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

然后，您可以向已提交的文件更改（按钮Reply）或整个提交 添加注释

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

由此产生的沟通就是代码审查。但是，我个人建议尽可能在一台 PC 上进行代码审查并进行面对面的交流，并使用工具来记录结果或需要更正式的形式。

对于有大量提交的文件评论，例如http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md ，请使用它blame来了解谁做了什么。然而，在此视图中，没有进行交流和添加评论的选项。在这种情况下，我建议只添加更改作为注释。