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:
- Petri-Dish +1 gives an error: needs code review Jeff Morriss (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Graham Bloice (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Evan Huus (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review mmann78 (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Gerald Combs (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Jeff Morriss (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Christopher Maynard (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Guy Harris (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Evan Huus (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Alexis La Goutte (Aug 16)
- Re: Petri-Dish +1 gives an error: needs code review Graham Bloice (Aug 16)
- Re: Petri-Dish +1 gives an error: needs code review Alexis La Goutte (Aug 16)
- Re: Petri-Dish +1 gives an error: needs code review Jeff Morriss (Aug 15)
- Re: Petri-Dish +1 gives an error: needs code review Graham Bloice (Aug 15)