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

Tom White updated WHIRR-21:
---------------------------

    Attachment: WHIRR-21.patch

This patch adds checkstyle checks (use mvn checkstyle:checkstyle to run) and 
fixes existing problems. Adds a build-tools module following the recommendation 
at 
http://maven.apache.org/plugins/maven-checkstyle-plugin/examples/multi-module-config.html.

Since this makes lots of whitespace changes (to fix errant tabs) I'm going to 
check it in soon.

> Enforce source code style
> -------------------------
>
>                 Key: WHIRR-21
>                 URL: https://issues.apache.org/jira/browse/WHIRR-21
>             Project: Whirr
>          Issue Type: Improvement
>            Reporter: Tom White
>             Fix For: 0.1.0
>
>         Attachments: WHIRR-21.patch
>
>
> We should use checkstyle to enforce the Java style guidelines. See 
> http://maven.apache.org/plugins/maven-checkstyle-plugin/usage.html.
> I suggest these should be same as Hadoop's (Sun's conventions except 2 spaces 
> for tabs, http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to