Tutorial marick #552

Merged
merged 2 commits into from May 1, 2012

Conversation

3 participants
Contributor

marick commented May 1, 2012

I had to look at Maven documentation for understanding first time I saw a dependency with a group-id, so I added an example.

This pull request passes (merged 3c2bf42 into e474ad2).

Collaborator

michaelklishin commented May 1, 2012

The line about omitting artifact group for Clojure projects is not a good idea in my opinion. It is also not very well justified: it's not a matter of Java vs Clojure but about namespacing, just like for functions. All my Clojure libraries have artifact groups, for the namespacing reason. Projects like Incanter, which have subcomponents, group components together using group ids.

@michaelklishin michaelklishin added a commit that referenced this pull request May 1, 2012

@michaelklishin michaelklishin Merge pull request #552 from marick/tutorial-marick
Tutorial marick
9b8195e

@michaelklishin michaelklishin merged commit 9b8195e into technomancy:master May 1, 2012

Collaborator

michaelklishin commented May 1, 2012

Thank you! If you don't mind, I will reword the line slightly to say that group ids are optional and used for namespacing.

Contributor

marick commented May 1, 2012

That would be swell. Since I am ignorant, I built my text off of the original.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment