<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 10, 2013 at 1:16 PM, Tomislav Plavcic <span dir="ltr"><<a href="mailto:tplavcic@gmail.com" target="_blank">tplavcic@gmail.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 text="#000000" bgcolor="#FFFFFF"><div class="im">
    <div>On 10.06.2013 18:09, Chris McMahon
      wrote:<br>
    </div>
    <br></div>
    O.K. Git now works for me - I made a mistake (of not reading
    tutorial) and didn't use "git review" but "git push" command... :(<div class="im"><br>
    </div><div class="im"><br></div></div></blockquote><div style>Great!</div><div style>  </div><div style>I see your commit at <a href="https://gerrit.wikimedia.org/r/#/c/67865/">https://gerrit.wikimedia.org/r/#/c/67865/</a> <br>
</div><div style><br></div><div style>Now I would like you to visit that page in gerrit, find the text box and the button for "Add reviewer".   Put "cmcmahon" in the text box (you should see auto-complete), click "Add reviewer", then put "zfilipin" too.  Add anyone else you'd like:  right now only Željko and I have privileges to merge code in the qa/ repository.  (But we'd like to change that pretty soon.) </div>
<div style><br></div><div style>When you add a reviewer, that person automatically gets email that there is a code change in gerrit for them to review.</div><div style><br></div><div style>Historically a big part of "Quality Assurance" for Wikipedia has relied on code review.  Code review is an important part of how we make software at WMF.  Unfortunately gerrit is not the most intuitive code review tool in the world, but we adopted gerrit for many good reasons.  It is worth learning. </div>
<div style>-Chris </div><div style><br></div><div style><br></div><div style><br></div></div><br></div></div>