Re: [nyphp-talk] Code Reviews

2010-01-05 Thread Peter Becker
Tom - Thanks for the clarification and not seen as argumentative (at least by me). Tom Melendez wrote: Hi Folks, We do have a style guide, defined and documented directory structure and naming convention. My objective(s) in the review is to ensure that conventions are being maintained (ea

Re: [nyphp-talk] Code Reviews

2010-01-05 Thread Tom Melendez
Hi Folks, > We do have a style guide, defined and documented directory structure and > naming convention.  My objective(s) in the review is to ensure that > conventions are being maintained (easy to get lax in and nothing that would > ever show up in testing), and that coders (and testers) stay at

Re: [nyphp-talk] Code Reviews

2010-01-04 Thread Peter Becker
Yeah, that's how I was looking at it and since I'm the only one other than QA and coders that will be there, I'm not concerned or interested in having anyone look bad.  Nothing more frustrating than getting into a religious argument when there's essentially nothing wrong.  Just a different way

Re: [nyphp-talk] Code Reviews

2010-01-04 Thread Justin Dearing
On Wed, Dec 30, 2009 at 11:18 AM, Tom Melendez wrote: > > 2. You find something wrong, then what? And what if it is not > "really" wrong (Jones uses a selection sort but sould have used an > insertion sort, and no one told him beforehand). Do we hold up the > project until the engs figure out w

Re: [nyphp-talk] Code Reviews

2010-01-04 Thread Peter Becker
Ben, David, Tom, Thanks for the insightful input.. Sorry for the delay in responding, but was gone over the holidays. My take away is that they can be useful if applied in a manner that avoids "pissing contests" between specific approaches, driven from above with specific objective(

Re: [nyphp-talk] Code Reviews

2009-12-30 Thread Tom Melendez
Hi Peter, On Tue, Dec 29, 2009 at 2:30 PM, Peter Becker wrote: > Looking to get some views (and best practices) on code reviews. I used to > work at IBM on their early version of Websphere (as UI designer, not coder) > where our group had code reviews on a regular basis. I'm now managing a > sm

Re: [nyphp-talk] Code Reviews

2009-12-30 Thread David Krings
On 12/29/2009 5:30 PM, Peter Becker wrote: Looking to get some views (and best practices) on code reviews. At my current work the devs do code reviews once a mod, project, or milestone is completed. The code review takes place before the peer review (show and tell to QA and TW) and other devs

Re: [nyphp-talk] Code Reviews

2009-12-29 Thread Ben Sgro
Hello Peter, At my current company we do both peer reviews and group code reviews. The group code reviews seem to have the best impact, as the peer reviews have naturally ceased to happen. Our group code reviews happen bi-monthly and during the time (usually an hour) developers working on dif

[nyphp-talk] Code Reviews

2009-12-29 Thread Peter Becker
Looking to get some views (and best practices) on code reviews. I used to work at IBM on their early version of Websphere (as UI designer, not coder) where our group had code reviews on a regular basis. I'm now managing a small dev team working on a new web site using Zend PHP/MySql and am curiou