nick-boss-tech commented on PR #5201: URL: https://github.com/apache/solr/pull/5201#issuecomment-6389252015
**Quick Review:** - License handling is incomplete. Both vendored JavaScript bundles reference missing license files. Precommit also fails :solr:webapp:rat for all six new Swagger UI resources. Add the required headers and upstream notices, then verify the packaged output. - Document the shipped explorer URL separately from the developer helper. - Replace SOLR-XXXXX and add the changelog. - Add a smoke test for the packaged explorer; current tests do not establish that it works. AI-generated. -- 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]
