GSoC improvements

Seb Ruiz ruiz at
Wed Mar 4 02:01:41 CET 2009

2009/3/4 Dan Meltzer <parallelgrapefruit at>:
> On Tue, Mar 3, 2009 at 7:25 PM, Lydia Pintscher <lydia at> wrote:
>> Heya folks,
>> I have been thinking about what we can improve this year wrt to the
>> code that gets written during GSoC. As last year was the first year we
>> had a lot of newcomers as students I think there are some things we
>> can improve this year on the coding side.
>> One big problem last year was that some students commited huge chunks
>> of code that only very few people or even no-one reviewed post commit.
>> So I guess we should make it very very clear that we want to see
>> smaller commits on a more regular basis.
>> Is there anything else that could be useful? What is your opinion on a
>> more formal code review pre or post commit?
> I feel like the mentor should be responsible for code review of the
> student.  I think we need to setup crucible to make this more doable
> (it's annoying to review code on the mailing list).  Whether or not it
> happens pre or post commit can probably be left up to the
> mentor/student pair, or perhaps reccomend pre-commit review for the
> first few commits, and then when the mentor feels confident that the
> student will a) maintain style and b) has a good idea of the overall
> amarok design, it could switch to post-commit review.

I think this sounds like a good idea, but it should be up to the
mentor to decide whether they do pre/post commit reviews.

> Dan,
>> Also for the former students: Is there anything on the coding side we
>> could have done that you think would have helped you?
>> Let's make GSoC 2009 rock at least as much as last year's!
>> Cheers
>> Lydia
>> --
>> Lydia Pintscher
>> Amarok community manager
>> - -
>> _______________________________________________
>> Amarok-devel mailing list
>> Amarok-devel at
> _______________________________________________
> Amarok-devel mailing list
> Amarok-devel at

Seb Ruiz

More information about the Amarok-devel mailing list