Re: Building the code with the CheckStyle

2008-04-30 Thread Willem Jiang
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, Wi

Re: Building the code with the CheckStyle

2008-04-30 Thread Hiram Chirino
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

Re: Building the code with the CheckStyle

2008-04-30 Thread Bruce Snyder
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 ou

Re: Building the code with the CheckStyle

2008-04-30 Thread Willem Jiang
ED] Sent: 30. april 2008 12:04 To: camel-dev@activemq.apache.org Subject: Re: Building the code with the CheckStyle I've suffered heaps of pain with servicemix and cxf as often the checkstyle rules only work with eclipse and I use IntelliJ So I'd be -1 from making these mandatory -

Re: Building the code with the CheckStyle

2008-04-30 Thread James Strachan
2008/4/30 Willem Jiang <[EMAIL PROTECTED]>: > Hi James, > > After googling with the "IntelliJ Checkstyle" , I found there are lots of > Checkstyle plugins for IDE[1]. > Maybe checkstyle-idea[2] is an option for you :) > > [1]http://checkstyle.sourceforge.net/ > [2]http://code.google.com/p/check

Re: Building the code with the CheckStyle

2008-04-30 Thread Willem Jiang
Hi James, After googling with the "IntelliJ Checkstyle" , I found there are lots of Checkstyle plugins for IDE[1]. Maybe checkstyle-idea[2] is an option for you :) [1]http://checkstyle.sourceforge.net/ [2]http://code.google.com/p/checkstyle-idea/ Willem James Strachan wrote: I've suffered

RE: Building the code with the CheckStyle

2008-04-30 Thread Claus Ibsen
hilsen Claus Ibsen .. Silverbullet Skovsgårdsvænget 21 8362 Hørning Tlf. +45 2962 7576 Web: www.silverbullet.dk -Original Message- From: James Strachan [mailto:[EMAIL PROTECTED] Sent: 30. april 2008 12:04 To: camel-dev@activemq.apache.org Subject: Re: Building the code with the CheckStyle I&

Re: Building the code with the CheckStyle

2008-04-30 Thread James Strachan
I've suffered heaps of pain with servicemix and cxf as often the checkstyle rules only work with eclipse and I use IntelliJ So I'd be -1 from making these mandatory - until we figure out an easy way to configure *both* eclipse and IntelliJ. Also I'm with Claus on the 80 character limitation which

Re: Building the code with the CheckStyle

2008-04-29 Thread Willem Jiang
ECTED] Sent: 30. april 2008 06:04 To: camel-dev@activemq.apache.org Subject: Building the code with the CheckStyle 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 hea

RE: Building the code with the CheckStyle

2008-04-29 Thread Claus Ibsen
lding the code with the CheckStyle 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

Building the code with the CheckStyle

2008-04-29 Thread Willem Jiang
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).