On Fri, Feb 25, 2011 at 11:07 PM, Ewan Mellor <ewan.mel...@eu.citrix.com> wrote: > This is all great stuff Jay, but it's going to get lost if it's just in > email. http://wiki.openstack.org/LifeWithBzrAndLaunchpad has some of this > on, but it's nothing like as clear and succinct as you've got it below.
Cheers. > Could someone break out a "how peer reviews work" wiki page, and put this > content on it? We really need these things to be nice clear steps so that > everyone knows what to expect. Sure. I don't want to step on anything termie was doing, though. Andy, did you have a link to the wiki page you were putting together about reviewing? I'll add some thoughts to that. -jay >> -----Original Message----- >> From: openstack-bounces+ewan.mellor=citrix....@lists.launchpad.net >> [mailto:openstack-bounces+ewan.mellor=citrix....@lists.launchpad.net] >> On Behalf Of Jay Pipes >> Sent: 25 February 2011 10:07 >> To: openstack@lists.launchpad.net >> Subject: [Openstack] Working on fixing code after a review? Please mark >> merge proposal Work In Progress! >> >> Hey all, >> >> The backlog on code reviews continues to mount: >> https://code.launchpad.net/nova/+activereviews >> >> One thing that would REALLY help reviewers is the following: >> >> If you receive one or more reviews that have asked for fixes to your >> branch (Needs Fixing), and you agree to these fixes, please do the >> following: >> >> * Write a quick comment on the merge proposal acknowledging the review >> and stating you are working on fixing the branch >> * Set the merge proposal status to Work In Progress >> >> When you do this, your branch gets removed from the list of merge >> proposals that reviewers need to review (the link above), which helps >> reviewers to identify which branches are truly in need of review and >> which branches are actually being worked on after reviews. >> >> After you make the changes to your branch, please do the following: >> >> * bzr push your changes to Launchpad >> * Go to your merge proposal and set the status back to Needs Review >> >> This will trigger: >> >> * Launchpad to regenerate the diff that is displayed for your branch >> * Launchpad to email all prior reviewers to let them know they need to >> re-review your branch >> >> Doing these simple steps will make reviewing dramatically easier; the >> side-effect of that is reviews will be quicker and we can get through >> this backlog. >> >> Thanks for listening, >> jay >> >> _______________________________________________ >> Mailing list: https://launchpad.net/~openstack >> Post to : openstack@lists.launchpad.net >> Unsubscribe : https://launchpad.net/~openstack >> More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : openstack@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp