<div>Good call Jay - I'll do my best to remember to do this!</div><div><br></div><div>Why are branches with unmerged pre-reqs showing up in that list?  If reviewers are working from that list, that just seems to be creating extra work, which you probably don't need!</div>
<div><br></div><div>Justin<br>
<br><br><div class="gmail_quote">On Fri, Feb 25, 2011 at 10:07 AM, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com">jaypipes@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hey all,<br>
<br>
The backlog on code reviews continues to mount:<br>
<a href="https://code.launchpad.net/nova/+activereviews" target="_blank">https://code.launchpad.net/nova/+activereviews</a><br>
<br>
One thing that would REALLY help reviewers is the following:<br>
<br>
If you receive one or more reviews that have asked for fixes to your<br>
branch (Needs Fixing), and you agree to these fixes, please do the<br>
following:<br>
<br>
* Write a quick comment on the merge proposal acknowledging the review<br>
and stating you are working on fixing the branch<br>
* Set the merge proposal status to Work In Progress<br>
<br>
When you do this, your branch gets removed from the list of merge<br>
proposals that reviewers need to review (the link above), which helps<br>
reviewers to identify which branches are truly in need of review and<br>
which branches are actually being worked on after reviews.<br>
<br>
After you make the changes to your branch, please do the following:<br>
<br>
* bzr push your changes to Launchpad<br>
* Go to your merge proposal and set the status back to Needs Review<br>
<br>
This will trigger:<br>
<br>
* Launchpad to regenerate the diff that is displayed for your branch<br>
* Launchpad to email all prior reviewers to let them know they need to<br>
re-review your branch<br>
<br>
Doing these simple steps will make reviewing dramatically easier; the<br>
side-effect of that is reviews will be quicker and we can get through<br>
this backlog.<br>
<br>
Thanks for listening,<br>
jay<br>
<br>
_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~openstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
</blockquote></div><br></div>