FYI, I just committed a buildingtools module (copying from CXF), so you
could run the Checkstyle with -Psourcecheck.
Willem
Hiram Chirino wrote:
agree with enabling via a profile.
On Wed, Apr 30, 2008 at 1:07 PM, Bruce Snyder <[EMAIL PROTECTED]> wrote:
On Tue, Apr 29, 2008 at 10:04 PM, Willem Jiang <[EMAIL PROTECTED]> wrote:
> Hi Team,
>
> I'd like to enable the CheckStyle when build the camel code.
>
> There are lots of benefits which we can get from CheckStyle:
> 1.Finding out if we are missing the ASF header in the java file.
> 2 Making our codes more consistent ( not including any tab character which
> will cause bad formate).
> 3.Excluding the useless import package.
> ...
>
> All you have to do is install the CheckStyle pluging in Eclipse , and it
> should be same with the Intellij :)
>
> Any thought?
-1 to making them mandatory. Bury them in a profile so that they can
be run optionally. We've had waaaaay too much pain from Checkstyle
being mandatory in SMX and CXF.
Bruce
--
perl -e 'print unpack("u30","D0G)[EMAIL
PROTECTED]&5R\"F)R=6-E+G-N>61E<D\!G;6%I;\"YC;VT*"
);'
Apache ActiveMQ - http://activemq.org/
Apache Camel - http://activemq.org/camel/
Apache ServiceMix - http://servicemix.org/
Apache Geronimo - http://geronimo.apache.org/
Blog: http://bruceblog.org/
