[Insight-developers] Global Code Reviews

[Bill Lorensen]

I have not read the process yet, but why move the Review directory into an
external module. This will be very disruptive to Slicer4 and the Wiki
examples. What is wrong with leaving the Review where it is?

On Tue, Apr 26, 2011 at 1:12 PM, Luis Ibanez <luis.ibanez at kitware.com>wrote:

> As we discussed during the Boston meeting, a global
> code review effort will take place before the Beta release.
>
> The code assignments to contractors that Jim created,
> have been posted at:
>
> https://spreadsheets.google.com/a/kitware.com/spreadsheet/ccc?key=t8spmfVf1nHiKKkNnVunkUQ&authkey=CM-C2vcL#gid=0
>
>
> We are organizing the procedure for the code reviews in:
> http://www.itk.org/Wiki/ITK_Release_4/Global_Code_Review
>
>
> One of the first action items will be to move the Review
> directory out of the repository and make it available as
> an External module.
>
>
> Then we will identify the classes from Review that are
> commonly used (e.g. in applications like Slicer), and will
> put those classes in the top priorities of code review to
> move them into their final destination inside of the toolkit.
>
>
>   Please share your thoughts on this process,
>
>
>       Thanks
>
>
>            Luis
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.html
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110426/4a156ba2/attachment.htm>