slawekjaranowski commented on a change in pull request #52: URL: https://github.com/apache/maven-dependency-analyzer/pull/52#discussion_r811632962
########## File path: src/main/java/org/apache/maven/shared/dependency/analyzer/DefaultProjectDependencyAnalyzer.java ########## @@ -33,9 +33,11 @@ import java.util.LinkedHashMap; import java.util.LinkedHashSet; import java.util.Map; +import java.util.Optional; Review comment: https://maven.apache.org/developers/compatibility-plan.html Since June 2020 Java 8 is used in Maven plugins ... so we don't stop it Now I don't use Optional ... of course it is some type of taste, one likes Optional and other likes standard if 😄 -- 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-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org