> On 13 Jan 2016, at 18:37, Ben Coman <btc@openinworld.com> wrote:
>
> On Thu, Jan 14, 2016 at 12:22 AM, David Allouche <david@allouche.net> wrote:
>>
>> By the way, how do you guys do code reviews for code integrated into the
>> core?
>
> Its fairly informal.�� Once a Slice has been submitted to the
> Pharo5Inbox the associated Issue on Fogbugz is <Resolved> to "Fix
> review Needed" - people review it out and comment.�� Unfortunately with
> limited resources, sometime resolved issues can languish if reviewer
> don't feel its within their expertise, and often it falls the to small
> group of the core-devs from Inria.�� ��As Pharo grows more popular we
> may need some improved processes to reduce load on core-devs.

I believe that is an area that needs improving. But probably not right now.

>�� I'd be
> interested in what you've seen work in other environments.

Unfortunately, I do not have a lot of direct and relevant experience there. We did code reviews at Canonical, but I would never apply the kind of process we used there in a community project.

I do believe it is essential that the path from "someone submits code" and "send code review" be as frictionless as possible. Code reviews are hard and time consuming. Any time spent not actually reading and commenting code is a put off.

Concretely, that means I think a good, interactive, actively maintained, web-based code review tool is essential. I heard good things about github. But since people in this community seem to have a strong bias towards doing everything in Pharo, that might not be such a good idea...

In any case, I think it should be easy to review code without loading the change in the image.

>
> Have you seen these...
> http://pharo.org/contribute-propose-fix

I have seen it. But I did not feel that was relevant to me: I was not yet familiar with any of tools mentioned there, nor was I familiar with the policies surrounding code contributions.

> https://pharo.fogbugz.com/?W68

I had seen it too, when exploring the issue tracker. And similarly, I felt it was not yet relevant to me. So I quickly forgot about it :-)

I will be sure to refer to those documents as my involvement and confidence increase.

Regards.