On Tue, Nov 1, 2011 at 9:51 AM, Fournier, Camille F. <camille.fourn...@gs.com> wrote: > Committers, this checking in of code without tests has got to stop. It's > actively detrimental to the code base and undermines any value we might be > getting from refactoring efforts. With very limited exceptions (changing > socket parameters or something else that is virtually impossible to test > without significant investment in additional libraries and scaffolding), we > should not put in ANY more changes without tests. There's a reason we see a > -1 in the Hudson build report for no new tests. Don't take it lightly.
There's a difference btw checking in refactorings and fixing warnings and such that are covered by existing test, vs committing feature changes and bug fixes that are not caught by existing tests. Patrick