[
https://issues.apache.org/jira/browse/RANGER-2725?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17040147#comment-17040147
]
Lars Francke commented on RANGER-2725:
--------------------------------------
You can see my versions of those two in my patch:
{code:java}
<module name="FileTabCharacter"> <!--
https://checkstyle.org/config_whitespace.html#FileTabCharacter -->
<property name="eachLine" value="true"/>
</module>
<module name="Indentation"/> <!--
https://checkstyle.org/config_misc.html#Indentation -->
{code}
What have you got? I'm happy to merge them.
> Improve the Checkstyle integration from RANGER-2588
> ---------------------------------------------------
>
> Key: RANGER-2725
> URL: https://issues.apache.org/jira/browse/RANGER-2725
> Project: Ranger
> Issue Type: Improvement
> Components: Ranger
> Reporter: Lars Francke
> Assignee: Lars Francke
> Priority: Minor
> Attachments:
> 0001-RANGER-2725-Improve-the-Checkstyle-integration-from-.patch,
> RANGER-2725.1.patch, RANGER-2725.2.patch
>
>
> RANGER-2588 Added a basic checkstyle configuration.
> This moves the versions to newer ones (the original ones were from 2015) and
> adds a bunch of rules. I went through the Sun and Google code styles as well
> as the new proposed OpenJDK ones and picked a bunch of rules that make sense
> to me.
> I avoided adding rules that validate names of anything (e.g. classes and so
> on).
> This will generate a lot of warnings but...well...that just needs cleaning up
> :)
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
