dweiss commented on issue #14257:
URL: https://github.com/apache/lucene/issues/14257#issuecomment-2755678098

   Here is what I did.
   * added a brute-force non-formatting to any /// line comments in my fork of 
google-java-format [1]
   * added a local, precompiled binary of the above to my fork of Lucene's 
repository and modified spotless to use it. [2][3]
   
   It seems to work (I've modified one of the classes, intentionally leaving a 
super-long line there). This is just a PoC that it's doable... I'm not sure if 
this patch would be accepted to google-java-format as is (it's makes an 
assumption that anything starting with /// should just be left alone). I also 
don't think we can store a binary blob of google java format in Lucene 
repository. 
   
   I can try to initiate a discussion with google-java-format folks first. If 
this doesn't work, I can publish my fork under my own coordinates to Maven 
Central - then we won't need the local binary blob and things should just work.
   
   [1] 
https://github.com/google/google-java-format/compare/master...dweiss:google-java-format:preserve-markdown-like-lines
   [2] 
https://github.com/apache/lucene/compare/main...dweiss:lucene:gjf-markdown-friendly?expand=1
   [3] https://github.com/dweiss/lucene/tree/gjf-markdown-friendly
   
   


-- 
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: issues-unsubscr...@lucene.apache.org

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