[GitHub] [maven-dependency-plugin] rfscholte commented on issue #24: [MDEP-435] Added xml outputType to dependency tree

[GitBox <gi...@apache.org>]

rfscholte commented on issue #24: [MDEP-435] Added xml outputType to dependency tree
URL: https://github.com/apache/maven-dependency-plugin/pull/24#issuecomment-552093756
 
 
   To improve this patch:
   - try to write al least tests to confirm this works (and to prevent regression in the future due to other changes)
   - I personally would go for a different style of XML, maybe one that looks more like the pom. At least I would expect an explicit artifactId. the element should have the dependency-tag.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services