-Re: Maven Build Checkstyle
Jacques Nadeau 2013-08-11, 18:36
Thanks and welcome to the community!
For the output: yikes... guess that means we have a little work to do.
I have requested on INFRA the addition of a target version field. I
think this and DRILL-165 are required to be target to the alpha (0.1)
On Fri, Aug 9, 2013 at 10:25 AM, Nyoman Ribeka <[EMAIL PROTECTED]> wrote:
> Hi everyone,
> My name is Win (from my middle name Winardi). I was in the Apache Drill
> presentation at Oscon. I really like the presentation and I would like to
> contribute to the project. :)
> So I went through the Drill's JIRA and come to DRILL-134.
> Attached is my take on the issue. The patch will add a sub-module
> "build-tools" where the checkstyle configuration will be stored. This
> approach I think will allow all other sub-module to inherit the same
> configuration without the need to duplicate the same configuration into
> every sub-module.
> The patch will stop the maven build process when there are codes that
> doesn't conform to the coding standard (defined inside
> drill-checkstyle.xml). The checkstyle configuration is based on the
> sun_checks.xml provided by default in the checkstyle distribution.
> We can make the checkstyle to not-fail on checkstyle violation by changing
> the goal from "check" to "checkstyle". We can also change the configuration
> if we feel the configuration is too strict.
> PS: Attached is the checkstyle result on the prototype/common sub-module :)
>  https://issues.apache.org/jira/browse/DRILL-134
> -Nyoman Ribeka