[GitHub] [incubator-paimon] zhuangchong commented on pull request #926: [Improve] remove checkstyle and scalastyle check

["zhuangchong (via GitHub)" <gi...@apache.org>]

zhuangchong commented on PR #926:
URL: https://github.com/apache/incubator-paimon/pull/926#issuecomment-1512361497

   > > checkstyle should not be removed. Currently, the project uses spotless format and checkstyle together. For example: `import *`, `shade use`, etc. in checkstyle
   > 
   > https://www.libhunt.com/compare-spotless-vs-checkstyle?ref=compare
   > 
   > I think spotless and checkstyle are duplicate checks, we should not use them at the same time, If spotless doesn't support these checks, what we should do is make it support these checks.
   
   When removing checkstyle, can you add the project checkstyle special configuration item to the spotless format?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org