I use the same rules as the maven plugin, you must just import the settings.
The problem is that maven-checkstyle-plugin does not check the test-src but
eclipse does (per default).

So what do you think? should we fix this errors or should the developers
exclude test classes?

2012/9/15 Joseph Bergmark <bergm...@apache.org>

> I haven't used the checkstyle plugin for Eclipse, so afraid I can't
> provide much advice here.  Its too bad it doesn't appear to follow the
> same rules as the maven plugin.
>
> Sincerely,
>
> Joe
>
> On Sat, Sep 15, 2012 at 1:15 PM, Thomas Andraschko <zoi...@gmail.com>
> wrote:
> > It's only in test-classes because the won't be checked by maven.
> > For example -> AbstractUnitTest#addExtension
> >
> > 2012/9/15 Joseph Bergmark <bergm...@apache.org>
> >
> >> Which test classes are you getting check style errors in?  When I
> >> build on the command line I don't see any check style errors:
> >>
> >> [INFO] --- maven-checkstyle-plugin:2.7:check (verify-style) @
> >> openwebbeans-web ---
> >> [INFO]
> >> [INFO]
> >>
> >>
> >> [INFO] --- maven-checkstyle-plugin:2.7:check (verify-style) @
> >> openwebbeans-impl ---
> >> [INFO]
> >> [INFO]
> >>
> >> etc..
> >>
> >> Sincerely,
> >>
> >> Joe
> >>
> >> On Sat, Sep 15, 2012 at 12:26 PM, Thomas Andraschko <zoi...@gmail.com>
> >> wrote:
> >> > Hi,
> >> >
> >> > we have many checkstyle errors (not warnings) in the test classes.
> >> > Can i fix them or should i exclude (don't know if possible in Eclipse)
> >> the
> >> > test classes from checkstyle?
> >> > The most errors are "{ should be on a new line" and naming errors of
> >> > constants and variables.
> >> >
> >> > Also i have many eclipse errors that @PostConstruct, @PreDestory,
> >> > @WebServiceRef etc are not accessible.
> >> > Can we add this APIs with provided scope or is there any better
> solution?
> >> >
> >> > Regards,
> >> > Thomas
> >>
>

Reply via email to