# Code review tool recommendations - replacement for crucible?

3 messages from 2016-06-16 to 2016-06-16. Participants: Andreas Krey, Richard Ipsum.
Thread: https://gitlist.dev/t/42627

## Andreas Krey, 2016-06-16 09:41

Subject: Code review tool recommendations - replacement for crucible?
Message-ID: <20160616094108.GA14661@inner.h.apk.li>
URL: https://gitlist.dev/e/20160616094108.GA14661%40inner.h.apk.li

```
Hi all,

I'm looking for pointers to review tools that work with git (obviously),
and can deal sensibly with bigger reviews. Things we need:

- Ability to split (set of) commits into multiple reviews,
  so parts of changes can be reviewed by the respective owners
  (or assign different reviewers to different files/subtrees
  in a review).

- Tracking of files (or changes) already reviewed (due to the large numbers),
  and of the handling of issues found so far.

- Support incremental reviews, not just e.g. over the content of a
  pull request (bitbucket) When review comments are processed people
  want to be able to only review that change, and not to be forced
  to find that change in the entire previous changeset without
  the tool's support.

We were mostly content with atlassian crucible so far, but
it simply fails to index[1] our large product repo (5+ Gb) so
we switched to just give it diffs to review, and it fails
to properly display renames, and fails in a few minor
but annoying ways in dealing with these big reviews.

Any pointers?

- Andreas

[1] Crucible seems to be svn-centric, and the mapping
    from git to svn changesets they use internally
    apparently is O(n^{too much}) - just indexing
    a new branch identical to an existing one takes
    hours here.

-- 
"Totally trivial. Famous last words."
From: Linus Torvalds <torvalds@*.org>
Date: Fri, 22 Jan 2010 07:29:21 -0800

```

## Richard Ipsum, 2016-06-16 10:55

Subject: Re: Code review tool recommendations - replacement for crucible?
Message-ID: <20160616105556.GA8721@salo>
URL: https://gitlist.dev/e/20160616105556.GA8721%40salo
In-Reply-To: <20160616094108.GA14661@inner.h.apk.li>

```
On Thu, Jun 16, 2016 at 11:41:08AM +0200, Andreas Krey wrote:
> Hi all,
> 
> I'm looking for pointers to review tools that work with git (obviously),
> and can deal sensibly with bigger reviews. Things we need:
> 
> - Ability to split (set of) commits into multiple reviews,
>   so parts of changes can be reviewed by the respective owners
>   (or assign different reviewers to different files/subtrees
>   in a review).
> 
> - Tracking of files (or changes) already reviewed (due to the large numbers),
>   and of the handling of issues found so far.
> 
> - Support incremental reviews, not just e.g. over the content of a
>   pull request (bitbucket) When review comments are processed people
>   want to be able to only review that change, and not to be forced
>   to find that change in the entire previous changeset without
>   the tool's support.
> 

Hey,

Have you considered Gerrit[1] already?
It would seem to handle the cases you're interested in.

[1]: https://gerrit-review.googlesource.com/Documentation/

```

## Andreas Krey, 2016-06-16 12:14

Subject: Re: Code review tool recommendations - replacement for crucible?
Message-ID: <20160616121419.GQ30134@inner.h.apk.li>
URL: https://gitlist.dev/e/20160616121419.GQ30134%40inner.h.apk.li
In-Reply-To: <20160616105556.GA8721@salo>

```
On Thu, 16 Jun 2016 11:55:56 +0000, Richard Ipsum wrote:
...
> Have you considered Gerrit[1] already?
> It would seem to handle the cases you're interested in.

Possible, but only after a lot of user education.

We don't currently rewrite commits for review comments, and
neither can we get (all) people to create feature-oriented
commits, so we really need to review multiple commits in
one review, and assign reviewers to subtrees. We could do
that by repacking the changes into new commits, but that
would defeat the purpose.

Also, we often do reviews after feature integration,
and also do partial reviews long before a new
project is integrated.

In other words: Lots of impedance mismatch.

Andreas

-- 
"Totally trivial. Famous last words."
From: Linus Torvalds <torvalds@*.org>
Date: Fri, 22 Jan 2010 07:29:21 -0800

```