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

Reply via email to