morrySnow commented on PR #25033: URL: https://github.com/apache/doris/pull/25033#issuecomment-1754305483
checkstyle is not just for code format check. it is also check some code logic check. for example, - ensure no fall through branch in switch statement - ensure switch statement has default branch - name name overload method should put togather - must have java doc for long method - ... does spotless do same check now? -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
