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

Nick Dimiduk commented on PHOENIX-1962:
---------------------------------------

Yes, we are checking for unused imports as well. Here's the import-related 
section from the config file:

{noformat}
    <!-- Checks for imports                              -->
    <!-- See http://checkstyle.sf.net/config_import.html -->
    <module name="RedundantImport"/>
      <!-- Import should be explicit, and only from pure java packages.
           But we allow imports that are only used in javadoc. -->
    <module name="UnusedImports">
      <property name="processJavadoc" value="true"/>
    </module>
    <module name="IllegalImport" />
    <module name="AvoidStarImport" />
    <module name="ImportOrder">
      <property name="groups" 
value="java,javax,lib,shared,common,platform,org,com,io,net,scala,clover"/>
      <property name="ordered" value="true"/>
      <property name="separated" value="true"/>
      <property name="option" value="bottom"/>
    </module>
{noformat}

> Apply check style to the build
> ------------------------------
>
>                 Key: PHOENIX-1962
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1962
>             Project: Phoenix
>          Issue Type: Task
>            Reporter: Nick Dimiduk
>            Assignee: Nick Dimiduk
>         Attachments: PHOENIX-1962.00.patch, PHOENIX-1962.01.patch
>
>
> Let's enforce our code style and structure at compile time in maven instead 
> of via IDE integration that isn't applied universally.



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

Reply via email to