[Discuss] php dev's code with warnings and notices
Peter (peabo) Olson
peabo at peabo.com
Mon Jul 28 01:45:23 EDT 2014
On July 27, 2014 at 2:26 PM Rich Braun <richb at pioneer.ci.net> wrote:
> Eric Chadbourne asked:
> > The code they have submitted works,
> > but it has a bunch of warnings and notices in the logs. I personally
> > think this is sloppy coding. My question is, how strong a stand should
> > I take on this issue? I have the senior role but I am also the "new
> > guy".
[ stuff omitted]
> * Implement a unit-test code coverage tool that reports all lines that aren't
> covered by unit tests, if you don't have that in place already, and run it at
> least daily if not every build;
> * Get the test coverage well above 50%;
> * Block all code check-ins that worsen the problem (i.e. implement a
> static-analysis gate).
What is a static-analysis gate? (Routine search revealed nothing.)
peabo
More information about the Discuss
mailing list