openstack team mailing list archive
Mailing list archive
Re: Working on fixing code after a review? Please mark merge proposal Work In Progress!
On Fri, Feb 25, 2011 at 11:07 PM, Ewan Mellor <Ewan.Mellor@xxxxxxxxxxxxx> 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.
> 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.
>> -----Original Message-----
>> From: openstack-bounces+ewan.mellor=citrix.com@xxxxxxxxxxxxxxxxxxx
>> On Behalf Of Jay Pipes
>> Sent: 25 February 2011 10:07
>> To: openstack@xxxxxxxxxxxxxxxxxxx
>> 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:
>> 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
>> * 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,
>> Mailing list: https://launchpad.net/~openstack
>> Post to : openstack@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~openstack
>> More help : https://help.launchpad.net/ListHelp