<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 24, 2015 at 12:17 PM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Fri, Apr 24, 2015 at 8:42 AM, Russell Bryant<br>
<<a href="mailto:russell@russellbryant.net">russell@russellbryant.net</a>> wrote:<br>
> On Fri, Apr 24, 2015 at 8:31 AM, Joshua Colp <<a href="mailto:jcolp@digium.com">jcolp@digium.com</a>> wrote:<br>
>><br>
>> Olle E. Johansson wrote:<br>
>>><br>
>>> Playing around following Matt's wiki page on gerrit usage, I created<br>
>>> a team branch and did two commits. When pushing it with "git review<br>
>>> {branch}" only the last commit shows up.<br>
>>><br>
>>> Is that the way it's supposed to be? I thought the whole branch was<br>
>>> the review subject, not just a single commit.<br>
>><br>
>><br>
>> Gerrit works on a single commit (what it refers to as a patch set) that<br>
>> you want included into a specific branch. As a result you need to squash all<br>
>> commits into a single one, and if review feedback warrants further changes<br>
>> they also need to be squashed back into a single commit with the original<br>
>> changes. The single commit you post for review is what is reviewed and<br>
>> merged into the branch.<br>
><br>
><br>
> Gerrit can also work on a patch series, and tracks dependencies between<br>
> those patches.<br>
><br>
<br>
</div></div>I could be mistaken, but in the case where you have a series of<br>
patches, each patch is put up for review separately with its own<br>
Change-Id, correct?</blockquote><div><br></div><div>That's right, but it should also track that they are related and in a particular order.</div><div><br></div><div><a href="https://gerrit.asterisk.org/#/c/255">https://gerrit.asterisk.org/#/c/255</a></div><div><a href="https://gerrit.asterisk.org/#/c/256">https://gerrit.asterisk.org/#/c/256</a></div><div><br></div><div>See the "Related Changes" bit.  It shows the patches in order.</div><div><br></div><div>-- </div><div>Russell Bryant </div></div></div></div>