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) @
openwebbe
It's only in test-classes because the won't be checked by maven.
For example -> AbstractUnitTest#addExtension
2012/9/15 Joseph Bergmark
> 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-checkst
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 wrote:
> It's only in test-classes because the won't be ch
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 Berg
We can filter them http://checkstyle.sourceforge.net/config.html#Filters
Le 15 sept. 2012 20:09, "Thomas Andraschko" a écrit :
> 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 d
My opinion as a pretty much completely inactive contributor to OWB is that it
would be better to have the test classes follow the same style rules as the
main code, but I'm not prepared to help fix it.
thanks
david jencks
On Sep 15, 2012, at 11:09 AM, Thomas Andraschko wrote:
> I use the same
I would fix it, that's no problem.
We also have many compiler warnings because sometimes @Override is used for
implemented methods in test classes, which is not compatible to Java 1.5.
Could i create tickets for @Override and checkstyle errors in test-classes?
I would fix them for 1.1.7.
2012/9/
for checkstyle fix them if you activate it in mvn build otherwise i fear
some of us will not check it (i know i'm part of them)
about @Override: do we still target j5?
*Romain Manni-Bucau*
*Twitter: @rmannibucau*
*Blog: http://rmannibucau.wordpress.com*
2012/9/16 Thomas Andraschko
> I would
g
> Cc:
> Sent: Sunday, September 16, 2012 12:11 PM
> Subject: Re: Eclipse/Checkstyle errors
>
> I would fix it, that's no problem.
>
> We also have many compiler warnings because sometimes @Override is used for
> implemented methods in test classes, which is not compatib
012 12:15 PM
> Subject: Re: Eclipse/Checkstyle errors
>
> for checkstyle fix them if you activate it in mvn build otherwise i fear
> some of us will not check it (i know i'm part of them)
>
> about @Override: do we still target j5?
>
> *Romain Manni-Bucau
> From: Romain Manni-Bucau
> > To: dev@openwebbeans.apache.org
> > Cc:
> > Sent: Sunday, September 16, 2012 12:15 PM
> > Subject: Re: Eclipse/Checkstyle errors
> >
> > for checkstyle fix them if you activate it in mvn build otherwise i fear
> > some of us
WAS on mainframes.
> > Among those are big stock exchanges, banks and a big western defence
> pact...
> >
> >
> >
> >
> >
> > - Original Message -
> > > From: Romain Manni-Bucau
> > > To: dev@openwebbeans.apache.org
> > &
12 matches
Mail list logo