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

[Christophe Fergeau]

On Tue, Aug 01, 2017 at 09:24:42AM -0400, Frediano Ziglio wrote:
> > 
> > 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.

If patchwork were able to detect series which have been pushed, and hide
them, then this workflow would be perfectly fine.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170801/c03d0094/attachment.sig>