dweiss commented on pull request #1802:
URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683402017


   This looks really good already! My personal preference would be to keep the 
processing options inside the render javadoc file, rather than pushing them 
down to each project's build.gradle but it can definitely wait. Other than that 
I kept wondering if this should piggyback public/private scope changes to 
classes. Those weight classes in particular - making them private will have 
side-effects for other code (that may be doing an instanceof, for example).


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

Reply via email to