Re: Proposal: review branches (was Re: [vdagent-win PATCH v6 2/5] Initial rewrite of image conversion code)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Tue, Aug 01, 2017 at 07:51:37AM -0400, Frediano Ziglio wrote:
> > About the "consistently use Reviewed-by" this is already been proved
> > to be not possible in our team. We use patchwork but we can't say we
> > use consistently these replies, lot of the times they came in different
> > format.
> 
> I'm never looking at patchworks, so won't notice that missing r-b are
> causing issues :) Maybe it's possible to make a team effort to be more
> consistent here?
> 
> Christophe
> 

Yes and no, I think tools should help not slow down.
Take as example my recent "README" series. 6 patches quite small
and simple. Hit reply, "ack to series", send. That is. Following the
tool (patchwork) would require to reply to all patch mail, adding an
"Acked-by: Name Surname <email>" and send.

Somebody could argue that with the web PR on Gitlab you could just
hit "merge" and you don't even need to push the patches manually.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]