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/
>
--
Regards,
Hiram
Blog: http://hiramchirino.com
Open Source SOA
http://open.iona.com
