slawekjaranowski commented on code in PR #271:
URL: https://github.com/apache/maven-resolver/pull/271#discussion_r1146862457
##########
maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/visitor/DependencyGraphDumper.java:
##########
@@ -31,25 +31,27 @@
import org.eclipse.aether.util.graph.manager.DependencyManagerUtils;
import org.eclipse.aether.util.graph.transformer.ConflictResolver;
+import static java.util.Objects.requireNonNull;
+
/**
- * A dependency visitor that dumps the graph to the console.
+ * A dependency visitor that dumps the graph to any {@link Consumer<String>}.
Meant for diagnostic and testing, as
+ * it may output the graph to standard output, error or even some logging
interface.
+ *
+ * @since 1.9.8
*/
-public class ConsoleDependencyGraphDumper implements DependencyVisitor {
+public class DependencyGraphDumper implements DependencyVisitor {
Review Comment:
👍 - we have similar implementation in many places
--
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
