gus-asf commented on pull request #1965: URL: https://github.com/apache/lucene-solr/pull/1965#issuecomment-709404255
Well that was misleading. Rat is happy locally, not sure why github was blaming RulesTest's rat analysis. I did find a couple of unused imports, and realized that apparently `gradlew :lucene:check` doesn't run ecjLint as I had expected. Just for fun javadocs were running out of memory locally when I tried a full top level precommit which gives this lovely cryptic message: ``` * What went wrong: Execution failed for task ':lucene:demo:renderJavadoc'. > java.io.IOException: Stream Closed ``` (had to edit line 543 of render-javadoc.gradle to get it to work) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org