Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Wed, 6 May 2015 15:57:01 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12827059.1430774390000.24726.1430927821632@Atlassian.JIRA>
In-Reply-To: <JIRA.12827059.1430774390000@Atlassian.JIRA>
References: <JIRA.12827059.1430774390000@Atlassian.JIRA>
 <JIRA.12827059.1430774390917@arcas>
Subject: [jira] [Commented] (ACCUMULO-3771) Configure automatic java code
 style formatting
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/ACCUMULO-3771?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14530780#comment-14530780 ] 

ASF GitHub Bot commented on ACCUMULO-3771:
------------------------------------------

Github user keith-turner commented on the pull request:

    https://github.com/apache/accumulo/pull/35#issuecomment-99521744
  
    >  I modified the "Trailing whitespace" checkstyle rule ignore whitespace after an asterisk *.
    
    Thats nice, instead of completely removing the check


> Configure automatic java code style formatting 
> -----------------------------------------------
>
>                 Key: ACCUMULO-3771
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3771
>             Project: Accumulo
>          Issue Type: Task
>    Affects Versions: 1.8.0
>            Reporter: Mike Walch
>            Assignee: Mike Walch
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> I configured automatic Java style formatting for Fluo in [FLUO-479|https://github.com/fluo-io/fluo/issues/479].  This was done using the maven-java-formatter-plugin which formats all Java classes on every build according to a specified Eclipse style configuration file.  This has worked well for Fluo so Christopher Tubbs asked me to create a patch for Accumulo.  
> This patch will be composed of two commits.  The first commit will be the configuration for the plugin.   The second commit will be all changes to Java classes after the plugin is run.
> While auto-formatting has been a nice addition for Fluo, it does require a lot of changes to the code initially.  If you are unhappy with this change, feel free to object...


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)