<div dir="ltr">Hi David,<div><br></div><div>From what I can tell, you can't even get the order of changesets in a topic in vanilla Gerrit. You can search by a topic but that seems to be it for topic support. Also, I don't think that there is any way of merging an entire topic. That's changeset based too.<br>

</div><div><br></div><div>So based on what I see, I disagree with "<span style="font-family:arial,sans-serif;font-size:13px"> </span><span style="font-family:arial,sans-serif;font-size:13px">topics in vanilla gerrit really aren't that bad" :-) If we were to use Gerrit, I think that we would have to require that branches are squashed to 1 commit except unusual circumstances.</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">So a clarification to what I meant by vanilla Gerrit workflow: for the most part, the workflow would require branches of single or at most a few commits. Longer running branches with 10s of commits would be a major pain to manage in Gerrit and would probably require special handling.</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">-berk</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Sat, Aug 23, 2014 at 10:18 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I'm all for sticking with gerrit.  I'll be sad to lose the topic<br>
review interface, but topics in vanilla gerrit really aren't that bad.<br>
We'll still be able to push topics and we'll be able to see all the<br>
changes that make up a topic.  I believe that what we lose is 1) the<br>
ability to do a topic-level review and 2) the ability to browse by<br>
topic.<br>
<br>
The only thing that annoys me with gerrit is that when you click<br>
"review", it hides all of the previous review comments until you're<br>
done. Hopefully the new gerrit has fixed this.<br>
<br>
The things I'd like to see in a code review system are:<br>
<br>
- Tight integration with the bugtracker.<br>
<br>
- Some kind of incentive for reviewers, even if it's just gold stars<br>
or a "top reviewer" list.<br>
<br>
Or we could be draconian and enforce a review/submit ratio, just like<br>
the old BBS's enforced upload/download ratios.<br>
<span class="HOEnZb"><font color="#888888"><br>
 - David<br>
</font></span></blockquote></div><br></div>