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


##########
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java:
##########
@@ -2967,6 +2975,7 @@ public boolean isApplicable() {
         @Override
         public void addProviderProperties() throws MojoExecutionException {
             convertTestNGParameters();
+            providerDeprecationWarning("TestNG provider is deprecated, will be 
removed in future versions of plugin.");

Review Comment:
   technically support will not be removed but only the specific provider code.
   So in fact users do not know they are using this provider as it's added 
automatically by the plugin.
   I feel strange to add a warning, it looks like telling users TestNG will not 
be supported anymore which is wrong.



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