gnodet commented on code in PR #612:
URL: https://github.com/apache/maven-jar-plugin/pull/612#discussion_r4110810803
##########
src/main/java/org/apache/maven/plugins/jar/TestJarMojo.java:
##########
@@ -35,9 +35,16 @@ public class TestJarMojo extends AbstractJarMojo {
/**
* Set this to {@code true} to bypass test-jar generation.
- * Its use is not recommended, but quite convenient on occasion.
+ *
+ * <p>Note: unlike {@code -Dmaven.test.skip=true} (which also skips test
compilation and makes
+ * the test-jar unavailable to downstream modules), this flag suppresses
only the packaging step
+ * while leaving test classes compiled and accessible within the reactor.
+ * Use {@code -DskipTests} or, with Maven 4.1+, {@code --skip-tests}
to skip test
+ * <em>execution</em> without preventing the test-jar from being built.</p>
+ *
+ * @since 2.2
*/
Review Comment:
Fixed in 7dee44e.
--
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]
