LoganZhuZzz commented on PR #17280:
URL: https://github.com/apache/kafka/pull/17280#issuecomment-2378305958

   > Thanks for the patch @LoganZhuZzz!
   > 
   > Left a few minor comments inline. Can you temporarily add a couple of 
files without a license so we can see the script working in this PR?
   
   Thank you for the feedback on the patch @mumrah 
   I have made the following changes:
   
   1. Temporarily added test files without licenses to validate the unapproved 
licenses summary in the GitHub workflow.
   2. Moved the "Annotate Rat errors" step below the "Annotate checkstyle 
errors" step for better clarity.
   3. Added a version check to ensure annotation steps run only on Java 21 to 
prevent duplicate reports.
   
   Please let me know if there’s anything else you would like me to address!


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

Reply via email to