merlimat opened a new pull request, #25458: URL: https://github.com/apache/pulsar/pull/25458
## Summary - Remove the checkstyle plugin and its configuration files (checkstyle.xml, suppressions.xml) - Expand the existing spotless configuration to cover: unused import detection, trailing whitespace removal, and license header validation - Update CI workflow to run only `spotlessCheck` instead of `checkstyleMain checkstyleTest` - Remove checkstyle-related workarounds in `pulsar-client-tools-test` This drops the semantic/linting checks that checkstyle provided (naming conventions, javadoc requirements, illegal imports, code quality checks like FallThrough/EmptyStatement) in favor of a lighter, auto-fixable approach via `./gradlew spotlessApply`. ## Motivation Spotless can auto-fix violations (`spotlessApply`) unlike checkstyle which only reports them. This reduces friction for contributors and CI cycle time. ## Test plan - [ ] `./gradlew spotlessCheck` passes with no violations - [ ] `./gradlew spotlessApply` produces minimal/no diffs on a clean checkout - [ ] CI build-and-license-check job succeeds -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
