[ 
https://issues.apache.org/jira/browse/MDEP-799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17727075#comment-17727075
 ] 

ASF GitHub Bot commented on MDEP-799:
-------------------------------------

MartinWitt commented on code in PR #325:
URL: 
https://github.com/apache/maven-dependency-plugin/pull/325#discussion_r1209218850


##########
src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java:
##########
@@ -380,6 +380,8 @@ public DependencyNodeVisitor 
getSerializingDependencyNodeVisitor(Writer writer)
             return new TGFDependencyNodeVisitor(writer);
         } else if ("dot".equals(outputType)) {
             return new DOTDependencyNodeVisitor(writer);
+        } else if ("json".equals(outputType)) {
+            return new JsonDependencyNodeVisitor(writer);
         } else {
             return new SerializingDependencyNodeVisitor(writer, 
toGraphTokens(tokens));

Review Comment:
   In the other PR someone asked for a switch case instead. Shall I convert 
this to a switch statement?





> improve mvn dependency:tree - add optional JSON output of the results
> ---------------------------------------------------------------------
>
>                 Key: MDEP-799
>                 URL: https://issues.apache.org/jira/browse/MDEP-799
>             Project: Maven Dependency Plugin
>          Issue Type: New Feature
>          Components: tree
>            Reporter: Zhenxu Ke
>            Priority: Major
>
> I'd like to add an output type JSON, will open a pull request soon



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to