mchades opened a new pull request, #9835:
URL: https://github.com/apache/gravitino/pull/9835
### What changes were proposed in this pull request?
Add a custom Spotless step to enforce the 100-character line length limit as
specified in the Google Java Style Guide.
Changes:
1. Add a custom Spotless step that checks line length (max 100 chars)
2. Use `ratchetFrom('origin/main')` for gradual adoption - only checks
changed files
3. Update `AGENTS.md` with Issue and PR guidelines
### Why are the changes needed?
Currently, Google Java Format (used by Spotless) does not enforce the
100-character line length limit. This means:
- `./gradlew build` passes even when lines exceed 100 characters
- Line length violations can only be caught during manual code review
This was discovered during PR review
([#9824](https://github.com/apache/gravitino/pull/9824#discussion_r2747481313)).
Fix: #9834
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
- Ran `./gradlew spotlessCheck` to verify the configuration works
- The `ratchetFrom` feature ensures existing code is not affected
- New/modified files will be checked against the 100-char limit
--
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]
