Tibor17 commented on a change in pull request #350:
URL: https://github.com/apache/maven-surefire/pull/350#discussion_r632103181



##########
File path: 
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
##########
@@ -4215,4 +4213,14 @@ private Classpath setCachedClasspath( @Nonnull String 
key, @Nonnull Set<Artifact
             return classpath;
         }
     }
+
+    /**
+     * No tests run failures.
+     */
+    enum NoTestsFailureReason

Review comment:
       I am also to 100% convinced about the names. It must depict the real 
meaning of usage in the code, as the one above. Feel free to propose better 
ones.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to