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

Update to dependency mapper #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

marcelbirkner
Copy link

Hi Gard,

I read your blog about this maven plugin and like the idea of storing project dependencies in a graph db like neo4j. I added a couple of cypher queries and examples to document possible use cases. I also updated the current dependency versions and made minor changes to the code, like added a createdAt property for each node.

Greetings,
Marcel

@gardleopard
Copy link
Contributor

Thank you Marcel!

We appreciate your pull request. The documentation you have written is excellent. However I can't merge the pull request because of the indentation changes. I added maven-checkstyle-plugin configurations on master to make the build fail when using tabs for indentation. Please use spaces for indentation and rebase against master, so I can more easily see your changes.

ilosamart added a commit to ilosamart/maven-dependency-mapper that referenced this pull request Apr 27, 2017
- Change requested in PR finn-no#1
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