BLU Discuss list archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Discuss] php dev's code with warnings and notices
- Subject: [Discuss] php dev's code with warnings and notices
- From: peabo at peabo.com (Peter (peabo) Olson)
- Date: Mon, 28 Jul 2014 01:45:23 -0400 (EDT)
- In-reply-to: <3cc0b1308f1bdc836aedf1b040d51ec5.squirrel@webmail.ci.net>
- References: <mailman.82751.1406467425.15487.discuss@blu.org> <3cc0b1308f1bdc836aedf1b040d51ec5.squirrel@webmail.ci.net>
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
- References:
- [Discuss] php dev's code with warnings and notices
- From: richb at pioneer.ci.net (Rich Braun)
- [Discuss] php dev's code with warnings and notices
- Prev by Date: [Discuss] Seeking information on binaries called "entities" and "fixup"
- Next by Date: [Discuss] Looking for WiFi router with certain characteristics
- Previous by thread: [Discuss] php dev's code with warnings and notices
- Next by thread: [Discuss] php dev's code with warnings and notices
- Index(es):