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

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

GitBox
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:
[hidden email]


With regards,
Apache Git Services