elharo commented on code in PR #3259:
URL: https://github.com/apache/maven-surefire/pull/3259#discussion_r2784263048


##########
maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java:
##########
@@ -558,6 +558,21 @@ public class IntegrationTestMojo extends 
AbstractSurefireMojo {
     @Parameter(property = "failsafe.excludeJUnit5Engines")
     private String[] excludeJUnit5Engines;
 
+    /**
+     * Warning about deprecated providers and features planed to remove in 
<code>3.6.x</code> plugin version.

Review Comment:
   planned to be removed



##########
maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java:
##########
@@ -558,6 +558,21 @@ public class IntegrationTestMojo extends 
AbstractSurefireMojo {
     @Parameter(property = "failsafe.excludeJUnit5Engines")
     private String[] excludeJUnit5Engines;
 
+    /**
+     * Warning about deprecated providers and features planed to remove in 
<code>3.6.x</code> plugin version.
+     * <br>
+     *
+     * Set to <code>false</code> to disable the warning if you want to stay on 
the <code>3.5.x</code> plugin version.
+     * <br>
+     *
+     * See the documentation for more details:<br>
+     * <a 
href="https://maven.apache.org/surefire/maven-failsafe-plugin/migration355.html";>Migration
 to 3.5.5 version</a>
+     *
+     * @since 3.5.5
+     */
+    @Parameter(property = "failsafe.providerDeprecationWarning", defaultValue 
= "true")

Review Comment:
   silencing warnings usually doesn't happen at the level of a maven property. 
It's a logging option. Too many switches and flags causes cognitive overload. 
At some point you need to make a decision.



-- 
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