Subham-KRLX opened a new pull request, #2505: URL: https://github.com/apache/sedona/pull/2505
Did you read the Contributor Guide? Yes, I have read the [Contributor Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/latest/community/develop/). Is this PR related to a ticket? Yes, and the PR name follows the format [GH-2189] Add Checkstyle pre-commit hook for Java. Closes #2189. What changes were proposed in this PR? This PR adds a Checkstyle pre-commit hook to automatically check Java code formatting and style consistency before each commit. The hook runs mvn checkstyle:check to detect style violations. Since Spotless already formats Scala code, no new hooks were added for Scala files. Key changes: Added Checkstyle configuration to .pre-commit-config.yaml Integrated it into the existing pre-commit workflow for Java sources Ensured compatibility with current Maven/Spotless setup Why are these changes needed? To improve and enforce Java code style consistency during development. Automating the style check helps contributors detect and fix formatting issues early, ensuring clean and maintainable code before review and CI builds. How was this patch tested? Installed pre-commit hooks locally and verified execution Ran pre-commit run --all-files to confirm the Checkstyle hook triggers correctly Intentionally introduced style violations to verify detection Confirmed normal commits proceed when no issues are found Does this PR include necessary documentation updates? No, this PR affects only development tooling and does not modify user-facing APIs or documentation. Does this PR introduce any user-facing changes? No user-facing changes. This only affects local development workflow. -- 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]
