Re: Checkstyle checks

[Stefan Seelmann <ma...@stefan-seelmann.de>]

Next project is our server. All checkstyle violations in ApacheDS are
fixed, so we are ready to enable the checks there too.

If there are no objections I'll release the checkstyle-configuration
(there is only a small change) and then enable the checkstyle plugin in
ApacheDS pom.

Kind Regards,
Stefan


On 07/20/2015 10:08 PM, Stefan Seelmann wrote:
> Hi all,
> 
> I tuned our checkstyle configuration [1] according to our code formatter
> and fixed lots of (sensible) violations in API.
> 
> I'd like to enable checkstyle checks for the API project and let the
> build fail on violations. Very same as we did for Kerby. Only the checks
> will be activated, no report generation in enabled because that is quite
> useless. Later other projects may follow.
> 
> For that I need to release the checkstyle-configuration [1] project
> first, then the our project parent POM.
> 
> Any objections or questions?
> 
> Kind Regards,
> Stefan
> 
> PS: of course we can tune the checkstyle configuration if you think
> particular rules make no sense.
> 
> [1]
> https://svn.apache.org/repos/asf/directory/buildtools/checkstyle-configuration/
> 

Re: Checkstyle checks

[Emmanuel Lécharny <el...@gmail.com>]

Le 27/09/15 00:56, Stefan Seelmann a écrit :
> Next project is our server. All checkstyle violations in ApacheDS are
> fixed, so we are ready to enable the checks there too.

Thanks for the hard work, Stefan !
>
> If there are no objections I'll release the checkstyle-configuration
> (there is only a small change) and then enable the checkstyle plugin in
> ApacheDS pom.
Please go ahead !