rmuir commented on PR #14812:
URL: https://github.com/apache/lucene/pull/14812#issuecomment-2986074641
This is just starting small. I really want to next clean up some of the rat
interaction in this file, especially given the changing APIs and
thread-unsafeness that makes the gradle code complex.
A lot of the logic around license/headers can be done here instead,
especially since it isn't a "legal" check (rat-sources does that). I already
use queries to match these headers in my editor and hide, e.g.:
```scheme
;; fold and initially close license headers
(program
(block_comment) @fold @foldclose
(#lua-match? @foldclose "^/[*%s]*Licensed to the Apache.*"))
```
We can do similar stuff with yaml rules and ensure licenses aren't javadoc,
come before package declaration, whatever all is going on here, and simplify
the build.
--
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
