[Openstack] Working on fixing code after a review? Please mark merge proposal Work In Progress!
Ewan.Mellor at eu.citrix.com
Sat Feb 26 04:07:40 UTC 2011
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.
> -----Original Message-----
> From: openstack-bounces+ewan.mellor=citrix.com at lists.launchpad.net
> [mailto:openstack-bounces+ewan.mellor=citrix.com at lists.launchpad.net]
> On Behalf Of Jay Pipes
> Sent: 25 February 2011 10:07
> To: openstack at 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:
> 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 at lists.launchpad.net
> Unsubscribe : https://launchpad.net/~openstack
> More help : https://help.launchpad.net/ListHelp
More information about the Openstack