[ 
https://issues.apache.org/jira/browse/OOZIE-1126?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

jun aoki updated OOZIE-1126:
----------------------------

    Attachment: OOZIE-1126.patch

Attaching a patch.
It is intended to
1. Check only tailing space, tab character, Apache Header, and line length (132 
characters). We can extend but it should be in another enhancment ticket.
2. Not to fail the build, no matter how invalid the code is (in fact the 
current code has a lot of tailing spaces and length violations) but shows 
warnings to console output and
3. Is executed upon mvn verify from the root.

Please review it.

                
> see if checkstyle works for oozie development.
> ----------------------------------------------
>
>                 Key: OOZIE-1126
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1126
>             Project: Oozie
>          Issue Type: Task
>          Components: build
>    Affects Versions: trunk
>            Reporter: jun aoki
>            Assignee: jun aoki
>            Priority: Minor
>             Fix For: trunk
>
>         Attachments: OOZIE-1126.patch
>
>
> Currently, some code style check is done at the precmomit build
> http://svn.apache.org/viewvc/oozie/trunk/bin/
> It can be done at local machine by maven checkstyle plugin, so that devs 
> would get much faster feedback comparing to waiting a precommit build 
> completes on Jenkins server.
> The original thread is here
> http://mail-archives.apache.org/mod_mbox/oozie-dev/201212.mbox/%3CCAFc2gLnff6EABP2EWXgqfexaTy2PJETNuUtenmVrYVhF8%2Ba0vg%40mail.gmail.com%3E

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to