ppkarwasz commented on code in PR #134:
URL: 
https://github.com/apache/logging-log4j-transform/pull/134#discussion_r1801839227


##########
log4j-transform-maven-plugin/src/main/java/org/apache/logging/log4j/transform/maven/LocationMojo.java:
##########
@@ -189,11 +192,17 @@ private WrappedIOException(IOException cause) {
         }
     }
 
-    private void validateLog4jVersion() throws MojoExecutionException {
-        Artifact log4jApi = project.getArtifacts().stream()
+    private boolean validateLog4jVersion() throws MojoExecutionException {

Review Comment:
   If the version of Log4j API is too old, we can log an error and skip the 
execution.



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