>>in container/openejb-core/pom.xml?
Yup, currently we have it hooked in 'openejb-core/pom.xml' and in 'pom.xml'
('style' profile). The existing 'style' profile is a dummy one. I will move
what we have in 'openejb-core/pom.xml' to the profile definition in the
main pom. I would like to use the same profile to hook the pmd also defined
by 'openejb-core/pom.xml'. The goal is to use the 'check-formatting' step
[Example:
http://ci.apache.org/builders/tomee-trunk-ubuntu/builds/112/steps/check-formatting]
for both checkstyle and pmd. It would make the build to break faster in
case we have styling issues.
>>BTW would be great to remove src/main/config/checkstyle.xml and use
new checktyle plugin inline feature (checkstyle.xml in <configuration>)
I prefer to have a separated configuration file for that. Anyway, the
current pmd checks already have a few custom configuration files.
Checkstyle would follow the same path.
[]s,
Thiago.
On Wed, May 21, 2014 at 11:26 AM, Romain Manni-Bucau
<[email protected]>wrote:
> Hi Thiago,
>
> in container/openejb-core/pom.xml?
>
> BTW would be great to remove src/main/config/checkstyle.xml and use new
> checktyle plugin inline feature (checkstyle.xml in <configuration>)
>
>
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau
>
>
> 2014-05-21 17:24 GMT+02:00 Thiago Veronezi <[email protected]>:
>
> > Hi guys,
> >
> > Where is the checkstyle trigger defined in our build? I would like to
> move
> > the existing openejb-core checkstyle and pmd checks to the root module. I
> > know how to do it for checkstyle, but I didnt find where we call 'mvn3 -U
> > checkstyle:checkstyle -Pstyle', so I can do the same thing for pmd.
> >
> > []s,
> > Thiago.
> >
>
