Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add showRedundant flag. #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hgschmie
Copy link

@hgschmie hgschmie commented Nov 1, 2015

To illustrate the actual code dependencies in a project (especially when
enforcing total dependency definition using e.g. the dependency plugin),
it is necessary to have all edges from projects to dependencies present.

Currently, in a dependency tree like this:

            A
           / \
          B   C
         /
        C

the graph plugin will only plot

           A
           |
           B
           |
           C

This pull request adds a flag that will not remove any "redundant"
edges, so the result will be

           A
          / \
         B   |
          \ /
           C

While these graphs are more cluttered, they are also very useful to
fully grok dependencies in large Maven projects.

To illustrate the actual code dependencies in a project (especially when
enforcing total dependency definition using e.g. the dependency plugin),
it is necessary to have all edges from projects to dependencies present.

Currently, in a dependency tree like this:

                A
               / \
              B   C
             /
            C

the graph plugin will only plot

               A
               |
               B
               |
               C

This pull request adds a flag that will not remove any "redundant"
edges, so the result will be

               A
              / \
             B   |
              \ /
               C

While these graphs are more cluttered, they are also very useful to
fully grok dependencies in large Maven projects.
@buildhive
Copy link

Jeff Caddel » graph-maven-plugin #3 SUCCESS
This pull request looks good
(what's this?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants