accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christopher <ctubb...@apache.org>
Subject Re: Heads up on new checkstyle rules in the build
Date Wed, 14 Jan 2015 22:56:55 GMT
I really wanted to set the checkstyle plugin to run at "validate", but due
to some features it supports, which require inspecting subclasses, it needs
to run post-compile. We may still be able to configure it to run at the
"package" phase, though. I have not looked into that. I've just been using
the default for the plugin, which is "verify".

The basic command I run is:

mvn verify

But, I also frequently run with any or all of the following:

-DskipTests
-DskipITs
-Dfindbugs.skip
-Dcheckstyle.skip

I'm pretty sure Jenkins is configured to run verify, so even if users
don't, Jenkins should catch stuff.



--
Christopher L Tubbs II
http://gravatar.com/ctubbsii

On Wed, Jan 14, 2015 at 5:03 PM, Josh Elser <josh.elser@gmail.com> wrote:

> As a part of the recent formatting that Christopher did (thanks again, for
> that), he also added the maven-checksyle-plugin to be sure we don't just
> commit a bunch of stuff that's re-broken.
>
> Be aware that the checkstyle plugin is bound to the verify lifecycle phase
> which means that a `mvn package` won't catch it (just like findbugs and rat
> have been operating). It will be a good thing to get in the habit of
> running something like `mvn verify -DskipITs` to make sure we don't break
> any of these rules.... (like I just did :D)
>
> - Josh
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message