Yashgoswami-ds commented on PR #4661: URL: https://github.com/apache/solr/pull/4661#issuecomment-5121753359
> This should not have passed the precommit check. `solr/core/src/test/org/apache/solr/handler/admin/api/GetNodeSystemInfoSolrCloudTest.java` is missing an ASF license, which is enforced by RAT which runs on `./gradlew check` @dsmiley Thanks for pointing this out. Just to clarify, the missing ASF license header was noticed after the initial CI run, and I added it along with the formatting fixes before the final merge. I'll make sure to check the full precommit/RAT results more carefully in future contributions so these issues are caught earlier. -- 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]
