jhaber commented on a change in pull request #20:
URL:
https://github.com/apache/maven-dependency-analyzer/pull/20#discussion_r465874512
##########
File path:
src/main/java/org/apache/maven/shared/dependency/analyzer/DependencyAnalyzer.java
##########
@@ -39,4 +39,7 @@
Set<String> analyze( URL url )
throws IOException;
+
+ Set<DependencyUsage> analyzeWithUsages( URL url )
Review comment:
In that case another option is to keep things as-is and have the
dependency plugin use the same pseudo-code as above, except it would catch
something like `NoSuchMethodError` instead of `UnsupportedOperationException`
(with a comment explaining the situation). Certainly not ideal, but it seems
like we ultimately need some way for the dependency plugin to detect that the
analyzer doesn't support the new functionality and gracefully degrade
----------------------------------------------------------------
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]
