slachiewicz opened a new pull request, #740:
URL: https://github.com/apache/maven-checkstyle-plugin/pull/740

   Fixes #738: MCHECKSTYLE-222-no-resources now asserts the report goal is 
skipped and that `target/reports/checkstyle.html` is absent, and MCHECKSTYLE-54 
asserts the plugin found and checked the sources. Also combines 
near-duplicates: the three MCHECKSTYLE-225 ITs become one reactor, and the 
MCHECKSTYLE-260 and MCHECKSTYLE-99 pairs become one IT each, keeping every 
assertion and adding a positive check where an IT only had a negative one.
   
   Verified: the 5 changed ITs plus 10 others pass with `-P run-its` (ITs 49 → 
45); breaking the production line behind each changed IT fails 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]

Reply via email to