mraible commented on PR #194: URL: https://github.com/apache/roller/pull/194#issuecomment-5847144842
Good idea, done. The apache-rat-plugin 0.18 is in the parent pom's pluginManagement, not bound to the build, so `mvn apache-rat:check` runs it, and a new "License headers" job in the workflow runs that check on every PR. RAT caught files my first pass missed, which now have headers: the weblogger ORM mappings, `persistence.xml`, `taglib.tld`, `context.xml-example`, the dummy test DB scripts, and the `docs/examples` code. It also rejected the shortened ASF notice seven test classes used, so those have the full header now. The exclusions are commented in the pom: prose docs (`*.md`, `*.adoc`), the bundled themes and the starter templates in `src/main/resources/themes`, SVGs, third-party code (`wait-for-it.sh`, jQuery Mobile's CSS, the old `jquery-2.1.1.min.js`), data files like the word lists and the OPML sample, and the generated JMeter plan. `apache-rat:check` passes on all three modules. -- 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]
