[Sugar-devel] changes to the review process (was Re: Sugar 0.88 packages)

[Tomeu Vizoso]

On Sat, May 15, 2010 at 00:02, Bernie Innocenti <bernie at codewiz.org> wrote:
> El Fri, 14-05-2010 a las 08:59 +0200, Tomeu Vizoso escribió:
>> I'm just asking for someone to propose a set of concrete and coherent
>> changes to the current process, is it really asking too much?
>>
>> I'm sorry but I cannot go through the old threads, ask individuals for
>> clarifications, then draft that new process myself.
>
> Do you want the proposal posted to the wiki?
>
> We could basically take Sasha's original plan and copy it to the wiki. I
> think it was quite well thought:
>
>  http://lists.sugarlabs.org/archive/sugar-devel/2010-April/023410.html
>
> I propose the following amendments:
>
> 1) Scrap the paragraph saying that any sugar developer can approve
> changes, because it turned out to be a controversial point. The "who"
> and the "where" of reviews are orthogonal topics that can be discussed
> independently.
>
> 2) Also scrap the part where Sascha proposes ways to track patches in
> the list, since we can now use Patchwork for this.
>
> 3) Add all the clarifications in my follow-up to Sascha's proposal:
>
>  http://lists.sugarlabs.org/archive/sugar-devel/2010-April/023468.html
>
> 4) The existing conventions still apply for referencing tickets in the
> commit logs
>
> 5) When a corresponding ticket exists, the committer should add a link
> to the patch discussion in Patchwork before committing the patch.
>
> Does this sound good enough for an initial iteration? If so, I could
> take care of transcribing it into the wiki. Then, we can further refine
> the process as we go.

Hi Bernie,

I think this is an excellent step forward, I really appreciate you
having taken the time to do this.

I still need to explore patchwork further and I'm a bit concerned
about requiring that reviewers amend the commit message but I'm
willing to give it a test drive and see how it works in real.

I'm looking forward for the new text proposal.

Regards,

Tomeu

> --
>   // Bernie Innocenti - http://codewiz.org/
>  \X/  Sugar Labs       - http://sugarlabs.org/
>
>