[
https://issues.apache.org/jira/browse/HDDS-3582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17107014#comment-17107014
]
maobaolong commented on HDDS-3582:
----------------------------------
[~elek] I think the checkstyle of Alluxio can be a case, it maintains well,
expect a rule that we must name the member of a class with the "m" prefix, but
many others rules are good.
https://github.com/Alluxio/alluxio/blob/master/build/checkstyle/alluxio_checks.xml
BTW, Alluxio add a intellij codestyle config file into repo, it make
contributor easy to format their code,
https://github.com/Alluxio/alluxio/blob/master/docs/resources/intellij/codestyles/AlluxioStyle.xml
> Update Checkstyle rule
> ----------------------
>
> Key: HDDS-3582
> URL: https://issues.apache.org/jira/browse/HDDS-3582
> Project: Hadoop Distributed Data Store
> Issue Type: Improvement
> Components: build
> Affects Versions: 0.6.0
> Reporter: maobaolong
> Priority: Major
>
> Now. the rules of checkstyle is hadoop style, as we all know, hadoop repo
> started 10 years ago, so some of its rules need to be update.
> For example.
> - 100 length characters of a line make more sense, because most of all have a
> better monitor than 10 years ago.
> - We need more style rules, such as, behind and after "{" and "}" should have
> a space.
> - We should have manage the import into group and order them
> hope our community getting better and better
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
