kgyrtkirk opened a new pull request, #19308:
URL: https://github.com/apache/druid/pull/19308

   Spotless could do a little bit more than checkstyle which is only pointing 
out where the problems are - it also  provides a way to fix issues.
   
   This changeset configures the importorder and some other simpler rules.
   I wanted to also configure the formatter - but that changes indentation in a 
lot of files; haven't figured out a way to avoid most changes which seem to be 
just noise.
   
   Even the import order changes a lot of files; there are a lot of files where 
a blank line between `javax` and `java` was missing.
   
   The real meaningfull changes could be viewed with:
   ```
   git diff `git merge-base HEAD apache/master` HEAD -w  --ignore-blank-lines
   ```
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to