And I asked for this merge :(
Agree that -2 is best way forward. Will ask csit-dev folks to follow this
practice too from now onwards.
> On 18 Oct 2016, at 18:51, Damjan Marion (damarion) <damar...@cisco.com> wrote:
> Hey Dave,
> It can happen to anybody. I’m the first one who will do the same.
> That’s why I’m suggesting that we stop that practice. “-2” is hard lock which
> will prevent merging it in until reviewer revokes it.
>> On 18 Oct 2016, at 19:29, Dave Wallace <dwallac...@gmail.com> wrote:
>> My bad -- sorry 'bout that. Not my best day at multi-tasking :-(
>> For those patches like the csit operational testing patch that I just
>> merged, I prefer -2 so everyone can see the status of tests and help out if
>> I agree with Ed, that transparency is very important for community
>> On 10/18/16 1:13 PM, Damjan Marion (damarion) wrote:
>>> We just got 1st DO_NOT_MERGE patch merged in.
>>> Can we going forward stop this practice, and use “-2” or Drafts instead?
>>> vpp-dev mailing list
>> vpp-dev mailing list
> vpp-dev mailing list
vpp-dev mailing list