[IGSTK-Developers] Code Reviews

[David Gobbi]

Oops!  I didn't mean to imply that the comments should be emailed,
just that the author should be informed by email.

Comments should go on the Wiki like before.

Luis Ibanez wrote:

> Once we start writing reviews, please CC the developers-list
> when we send comments to the authors.  In this way, all other
> developers will also learn from the comments.
>
> The Wiki page has now also space for the review comments,
> this time organized by class, so it will be easier for the
> authors to get the comments for their classes from a single
> place.
>
> http://public.kitware.com/IGSTKWIKI/index.php/Iteration_6_Code_Reviews
>
>
>
>   Luis
>
>
> -----------------------
> David Gobbi wrote:
>
>> Code review time again!
>>
>> There is a new, tidier code review page up for iteration 6:
>> http://public.kitware.com/IGSTKWIKI/index.php/Code_Reviews
>>
>> All active programmers plus BB, KG, SA should sign up to
>> review at least 2 or 3 classes, availability permitting.
>>
>> After a file has been reviewed, please email the author(s)
>> directly.  Authors should email the reviewers back as
>> soon as the reviewer's concerns have been addressed.
>> Ideally we want all code review communication to take
>> place over email, so that it doesn't eat up time at the
>> T-cons.
>>
>> - David
>>
>> _______________________________________________
>> IGSTK-Developers mailing list
>> IGSTK-Developers at public.kitware.com
>> http://public.kitware.com/cgi-bin/mailman/listinfo/igstk-developers
>>
>>
>
>