← Back to team overview

openstack team mailing list archive

Re: Marking 'Approved' with others who say 'Needs Fixing'

 

++

On Tue, Aug 23, 2011 at 10:06 AM, Brian Lamar <brian.lamar@xxxxxxxxxxxxx> wrote:
> I see this happening more and more when deadlines are coming up:
>
> There is a merge proposal which has 2+ Core Approvals and 1+ Core Needs Fixings and the branch is marked as 'Approved'. This is fine, in my opinion, if you've talked to the person and they have given verbal approval or if the 'Needs Fixing' review is stale by 48+ hours but this has been happening with basically fresh 'Needs Fixing' reviews.
>
> Most Core developers are going to be in IRC and don't mind being pinged if you really want to get something in, but in my opinion we should *not* be merging code when there are Core devs that have not completed their re-review after giving feedback.
>
> Pushing in things at the last minute before the deadline is NOT what we should be doing. If something needs to get in, we can make exceptions for those branches.
>
> ---
> Brian
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openstack
> Post to     : openstack@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openstack
> More help   : https://help.launchpad.net/ListHelp
>


References