hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-14829) Add more checkstyles
Date Wed, 18 Nov 2015 16:36:10 GMT

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

stack commented on HBASE-14829:
-------------------------------

The site build failed with this [~appy]

{code}
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.4:site (default-site)
on project hbase: Error generating maven-checkstyle-plugin:2.16:checkstyle: Failed during
checkstyle configuration: cannot initialize module TreeWalker - Property 'sortStaticImportsAlphabetically'
in module ImportOrder does not exist, please check the documentation -> [Help 1]
[ERROR] 
{code}



> Add more checkstyles
> --------------------
>
>                 Key: HBASE-14829
>                 URL: https://issues.apache.org/jira/browse/HBASE-14829
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Appy
>            Assignee: Appy
>         Attachments: HBASE-14829-master.patch
>
>
> This jira will add following checkstyles:
> [ImportOrder|http://checkstyle.sourceforge.net/config_imports.html#ImportOrder] : keep
imports in sorted order
> [LeftCurly|http://checkstyle.sourceforge.net/config_blocks.html#LeftCurly] : Placement
of left curly brace. Does 'eol' sounds right setting?
> [NeedBraces|http://checkstyle.sourceforge.net/config_blocks.html#NeedBraces] : braces
around code blocks
> [JavadocTagContinuationIndentation|http://checkstyle.sourceforge.net/config_javadoc.html#JavadocTagContinuationIndentation]
: Avoid weird indentations in javadocs
> [NonEmptyAtclauseDescription|http://checkstyle.sourceforge.net/config_javadoc.html#NonEmptyAtclauseDescription]
: We have sooooo many empty javadoc @ clauses. This'll take care of it.
>  
> [Indentation|http://checkstyle.sourceforge.net/config_misc.html#Indentation] : Bad indentation
hurts code readability. We have indentation guidelines, should be fine enforcing them.



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

Mime
View raw message