sheila miguez wrote: > He originally stated that he expected everyone to run every possible > combination of inputs before committing a change but later clarified > this and described a process with a limited set of tests that had to > pass before something is committed to the trunk. His first statement > was unreasonable. I objected. The clarification was reasonable.
We don't want to stifle development, but we need to make reasonably sure that changes don't break existing code, or if they do, that there's very good reason for it and that there's general agreement that the benefits outweigh the breakage of existing code. But, I'm the first to admit to being too casual about this up until now. For 2 years, Ed and I were really the only ones making significant changes, and we tended to find each other's bugs and generally agreed with each other's changes since we both have similar visions. There have been notable exceptions, of course. The thing is, we have no automated unit testing framework, save the tiny bit I've added in the past couple weeks. So we never really are sure if we aren't breaking something somewhere, no matter how much usability or acceptance testing we do against our various applications to make sure things work. We never will cover all cases, but we can cover all the common ones. The point is this, though, as we work to getting more automated testing in place and as we bring more developers on board (welcome again to Carl, John, Nate, Uwe, Simen, and also the users contributing needed feedback): While there are undoubtedly lots of things that can be made much better (including the good change that Carl made that instigated this thread), when making sweeping changes like that it needs to be thoroughly tested *and* we need to have general agreement from the other developers first (which would include, at the bare minimum, an implicit okay from either Ed or Paul, simply because we know almost like the backs of our hands what will likely cause trouble). -- pkm ~ http://paulmcnett.com _______________________________________________ Post Messages to: [email protected] Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-users
