Wireshark mailing list archives

Re: Petri-Dish +1 gives an error: needs code review


From: Graham Bloice <graham.bloice () trihedral com>
Date: Fri, 15 Aug 2014 20:36:06 +0100

On 15 August 2014 20:17, Jeff Morriss <jeff.morriss.ws () gmail com> wrote:

Hi folks,

One thing I've noticed about the Petri-Dish flag in Gerrit is that every
time I set it to +1 I get an error saying "needs code review."  Since Chris
just noticed and complained about that too (on change 3624) I thought I'd
mention it here.


I haven't seen that when I've set the Petri-dish flag.

I do have difficulty forcing a subsequent Petri-Dish build after the first
one though, does anyone know the incantations for that?

Also re. Chris's comment on 3624, I don't think we have auto-merge running
yet.  The item has to be at + 2 and then you can hit the Submit button.
There is no mandatory Petri-Dish flag yet.



-- 
Graham Bloice
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe

Current thread: