Forum OpenACS Q&A: Re: Quality of OpenACS

Collapse
16: Re: Quality of OpenACS (response to 1)
Posted by Nick Carroll on
I agree with Janine. Only working code should be committed to the CVS repository. Even code in HEAD should work, and not be completely broken.

I personally don't think the code reviewer suggestion will work. It works well within an office environment, where you can sit next to a person and review their code as they walk you through it. Not having the developer help walk you through the changes they've made will make it difficult to detect by the reviewer. Code reviews won't work in a distributed environment, unless we all have remote screen sharing software that we can use for this.