Hi Vinoth,
Here are some of my points: 1, When developers are not familiar with checkstyle rules, they feel uncomfortable. I think its a good idea to make the instructions on contributing guide work with the checkstyle rules we already have. 2, We can also prompt users in the PULL_REQUEST_TEMPLATE about how to check code style by themself manually. 3, The code of the current project has met the checkstyle rules, we just need handle the incremental codes. 4, here are some useful tools which can be placed on contributing guide. 1) mvn scalastyle:check 2) mvn checkstyle:check 3) https://checkstyle.sourceforge.io/index.html 4) http://www.scalastyle.org/rules-1.0.0.html best, lamber-ken On 12/23/2019 13:03，Vinoth Chandar<vin...@apache.org> wrote： Hello all, I know a bunch of work has happened to format the code base, closer to what other project are doing.. While working through some checkstyle violations today, I noticed that the IDE formatting is now out of date with the checkstyle enforced? Manually fixing these checkstyle issues are not a very productive use of time IMHO. http://hudi.apache.org/contributing.html#ide-setup Can we make the instructions here work with the checkstyle rules we already have? Goal should be that formatting code in IntelliJ (or other IDEs) should autofix it so that checkstyle passes.. thoughts? Thanks Vinoth
