[ https://issues.apache.org/jira/browse/RANGER-5017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17904370#comment-17904370 ]
Madhan Neethiraj commented on RANGER-5017: ------------------------------------------ {quote}how about using the latest version of Checkstyle and the plugin? {quote} [~bpatel] - any version that runs in JDK8 should be fine. Do you want to give it a shot? > Code readability improvement > ---------------------------- > > Key: RANGER-5017 > URL: https://issues.apache.org/jira/browse/RANGER-5017 > Project: Ranger > Issue Type: Improvement > Components: Ranger > Reporter: Madhan Neethiraj > Assignee: Madhan Neethiraj > Priority: Major > Fix For: 3.0.0 > > Attachments: RANGER-5017.patch > > > Ranger code base has been evolving over a decade, with contributions from > large number of authors. Having a common coding style enforced during build > time will help improve the readability of the code base. I propose to use > [checkstyle|https://github.com/airlift/airbase/blob/master/airbase-policy/src/main/resources/checkstyle/airbase-checks.xml] > based on airbase, which is used by open-source projects like Trino. > Updating the entire code base will take considerable time, hence this will > need to be done in phases, perhaps one module at a time. To start with, I > suggest updating the checkstyle currently in the master branch but not fail > the build in case of violations. This should be followed by updates to > sources to comply with the checkstyle - one module at a time. Once all > modules are updated, build should be updated to fail on violations. -- This message was sent by Atlassian Jira (v8.20.10#820010)