[
https://issues.apache.org/activemq/browse/SM-1275?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ulhas Bhole updated SM-1275:
----------------------------
Description:
My setup somehow pickup maven-checkstyle-plugin version 4.3. I have checked the
parent/pom.xml do not checkstyle plugin version locked to 4.1.
The errors reported are minor and easily fixable they are related to class
should be final, should not have public constructor and ordering of static
members. 2 classes in question are both utility classes only having static
members. I have a fix ready in my setup for checkstyle version 4.3 so I will
add the patch soon.
I am also including minor change in platforms/geronimo/deployer-service/pom.xml
to add exclusions dependencies that are not strictly required for this module.
1. axis2-jaxws-api
2. axis2-saaj-api
3. jstl
was:
My setup somehow pickup maven-checkstyle-plugin version 4.3. I have checked the
parent/pom.xml do not checkstyle plugin version locked to 4.1.
The errors reported are minor and easily fixable they are related to class
should be final, should not have public constructor and ordering of static
members. 2 classes in question are both utility classes only having static
members. I have a fix ready in my setup for checkstyle version 4.3 so I will
add the patch soon.
Also, even if I am fixing these errors I will also lock the version to current
checkstyle plugin version 4.1 in my patch.
> SMX parent pom do not have maven-checkstyle-plugin version locked so picking
> up checkstyle 4.3 version gives some checkstyle errors.
> -------------------------------------------------------------------------------------------------------------------------------------
>
> Key: SM-1275
> URL: https://issues.apache.org/activemq/browse/SM-1275
> Project: ServiceMix
> Issue Type: Bug
> Components: servicemix-core, servicemix-cxf-bc
> Affects Versions: 3.2
> Reporter: Ulhas Bhole
> Assignee: Ulhas Bhole
> Priority: Minor
> Fix For: 3.2.2
>
>
> My setup somehow pickup maven-checkstyle-plugin version 4.3. I have checked
> the parent/pom.xml do not checkstyle plugin version locked to 4.1.
> The errors reported are minor and easily fixable they are related to class
> should be final, should not have public constructor and ordering of static
> members. 2 classes in question are both utility classes only having static
> members. I have a fix ready in my setup for checkstyle version 4.3 so I will
> add the patch soon.
> I am also including minor change in
> platforms/geronimo/deployer-service/pom.xml to add exclusions dependencies
> that are not strictly required for this module.
> 1. axis2-jaxws-api
> 2. axis2-saaj-api
> 3. jstl
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
