Re: Building the code with the CheckStyle

[Willem Jiang <wi...@gmail.com>]

Hi Claus，

I used the Eclipse to find out the CS errors, you may need run mvn 
-Psetup.eclipse first to setup the CS rule first (It is 
trunk/etc/activemq-checkstyle.xml).
And you also need to install the CS plugin first.

You can find some information about how to set up eclipse here [1]

[1]http://cwiki.apache.org/CXF/setting-up-eclipse.html

Willem

Claus Ibsen wrote:
> Hi Willem
>
> +1
>
> I have wondered how you ran the checkstyle check since you have been checking and fixing this quite often on the code base.
>
> I wanted to be able to do it also before committing so the code was checked beforehand.
>
> Should we use a custom checkstyle rules? The default ones are a bit to greedy/oldish with eg. with max 80 char lines. This is kind late 80tish now that we all have widescreen laptops or 24" monitors with 1600x2000 
>
> The unused imports can IDEA also easility spot. And in fact it has a great library for all kind of code tests that can spot problems.
>
>
> Med venlig hilsen
>  
> Claus Ibsen
> ......................................
> Silverbullet
> Skovsgårdsvænget 21
> 8362 Hørning
> Tlf. +45 2962 7576
> Web: www.silverbullet.dk
> -----Original Message-----
> From: Willem Jiang [mailto:willem.jiang@gmail.com] 
> 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 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?
>
> Willem
>
>
>
>