slachiewicz opened a new pull request, #774: URL: https://github.com/apache/maven-pmd-plugin/pull/774
MPMD-317 and MPMD-318 become one IT with four invocations (plain, aggregate, and both with `includeTests`), whose verify script checks that test classes and test dependencies appear on the aux classpath only when `includeTests` is set. empty-rulesets gets real assertions: with `<rulesets/>` PMD runs no rules, so `pmd.xml` is written with no violations. Verified: `mvn -B verify -P run-its` → Passed: 41, Skipped: 4 (ITs 46 → 45); ignoring `includeTests` in either mode, or always including tests, fails the combined IT --- To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) check one of the following: - [ ] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0) - [x] I hereby declare that I have filed an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). -- 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]
