[Sugar-devel] Fwd: Proposal on how to speed up patch reviews

[Manuel Quiñones]

2013/3/26 Daniel Narvaez <dwnarvaez at gmail.com>:
> On 26 March 2013 12:21, Manuel Quiñones <manuq at laptop.org> wrote:
>> When I started in this project, my patches received reviews from many
>> people, not only maintainers.  Many discussions went by.  I don't see
>> that happening anymore.
>
> Could it be because most patches are posted in trac now? I'm not sure
> it's the only reason, though those are pretty much invisible to people
> not involved directly in the issue.

Right Daniel, could be.  At some point of the gtk3 port and touch
feature development it became a waste of efforts to bother the list
with our patches.  It was like a theatre play, Simon or me sending a
patch, the other ack-ing or nack-ing, no other actor intervening.  And
we wanted to speed up that transition.

But yes I'd favor going back to post the patches on the mailing list.
Or maybe better, discuss the tickets in the mailing list without
posting the patches here.  Could be any of both, as the sender
prefers.  I personally prefer to have my patches in the tracker.

> (I know there are issues with mailing list reviews too, just trying to
> analyze the situation here)
>
>> We also had periodic design meetings guiding
>> features.
>
> I wonder if that kind of discussion could be had on the mailing list
> rather than in meetings. It's problematic to find a time that works
> for everyone and many people just doesn't have time to spend in irc.

Yes after coordination issues we concluded we better keep discussing
design topics in the mailing list.  We have a prefix [DESIGN] for
this.  I will try to be more responsive to those threads from now on.

>> Discussing design after the development is made isn't good,
>> imho.
>
> I couldn't agree more.



-- 
.. manuq ..