This project has moved and is read-only. For the latest updates, please go here.

Code Review Process

Nov 4, 2009 at 7:47 PM

Just my input on the formulation of a code review process:  I think having a core team of reviewers for the actual nitty gritty code review would possibly be better than doing it on the weekly HIS calls.  Perhaps developers could submit/email the particular code files/modules they want reviewed to the review team on a weekly or biweekly basis.  The reviewers could then compile their comments and ideas for improvements, and maybe these comments could be made publicly available.

Nov 4, 2009 at 8:07 PM

I agree. Doing a full code review of the full project on today's call was a bit of an ambitious undertaking, and I think what we ended up with was basically an overview of the code structure with some examples. Perhaps as we move forward we can identify specific code modules and specific team members to provide a detailed review. For example when a new major function or plug-in is added. - Dan