[GitHub] [maven-dependency-plugin] S4n60w3n 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] S4n60w3n commented on issue #24: [MDEP-435] Added xml outputType to dependency tree

GitBox
S4n60w3n commented on issue #24: [MDEP-435] Added xml outputType to dependency tree
URL: https://github.com/apache/maven-dependency-plugin/pull/24#issuecomment-552326393
 
 
   > 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.
   @rfscholte Thanks for the comment.
   Do you mean something like this?
   ```
   <dependency>
         <groupId>org.apache.maven</groupId>
         <artifactId>maven-artifact</artifactId>
         <version>1.0.1</version>
        <dependencies>
            <dependency>
                <groupId>org.apache.maven</groupId>
                <artifactId>maven-model</artifactId>
                <version>1.0.0</version>
            </dependency>
        <dependencies>
   </dependency>
   ```
   

----------------------------------------------------------------
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