Re: [PR] [SUREFIRE-823] Decouple -DskipTests from Failsafe plugin [maven-surefire]

["Copilot (via GitHub)" <gi...@apache.org>]

Copilot commented on code in PR #3371:
URL: https://github.com/apache/maven-surefire/pull/3371#discussion_r3616800215


##########
maven-surefire-plugin/src/site/apt/examples/skipping-tests.apt.vm:
##########
@@ -88,9 +88,15 @@ mvn install -DskipTests
 mvn install -DskipTests
 +---+
 
+#{if}(${project.artifactId}=="maven-surefire-plugin")
  Since <<<skipTests>>> is also supported by the ${thatPlugin} Plugin, this will have the effect
- of not running any tests.  If, instead, you want to skip only the integration tests
- being run by the ${thisPlugin} Plugin, you would use the <<<skipITs>>> property instead:
+ of not running any tests. If, instead, you want to skip only the integration tests
+ being run by the ${thatPlugin} Plugin, you would use the <<<skipITs>>> property instead:

Review Comment:
   In the Surefire-plugin branch of this example, the text still claims that `-DskipTests` (aka `skipTests`) will also skip the Failsafe plugin, resulting in *no tests* running. After this PR’s behavioral change, `-DskipTests` should only skip Surefire unit tests and must not skip integration tests run by the Failsafe plugin, so this paragraph becomes misleading.



-- 
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-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org