I'm already frustrated by how long it takes the rat checks to run during build/test.
On Tue, Mar 10, 2009 at 10:51 AM, Paul Lindner <[email protected]> wrote: > checkstyle runs in the reporting profile, which means that it's not really > enforced. If we run it during the build/test phase then we can enforce > style. > > > > On Mar 10, 2009, at 9:51 AM, Vincent Siveton wrote: > > Paul, >> >> 2009/3/10 Paul Lindner <[email protected]>: >> >>> If we're going to be pedantic about these sorts of things we should >>> enforce >>> them with checkstyle. This way the checkin that activates the rule can >>> include all the fixes to get the codebase in conformance. >>> >> >> As I already mentioned in this thread, our Checkstyle conf [1] already >> checks for trailing space. >> >> Cheers, >> >> Vincent >> >> [1] >> http://svn.apache.org/repos/asf/incubator/shindig/trunk/site/checkstyle.xml >> > >

