Support nrepl/nrepl about clojure-maven-plugin HOT 12 CLOSED

Hey Daniel thanks for doing this and keep in mind I can help with the grunt work, now that you explained the problem I can apply the exclusions...

Oh ...if you found that's even better 😁

from clojure-maven-plugin.

The change is indeed trivial. The problem is that the test is breaking, and the reason for that is rather involved and related to how the integration tests are run. I can see that tools.nrepl is present on the classpath despite explicitly pulling nrepl/nrepl. I suspect a transitive dependency is pulling it in. I am currently trying to hunt it down. Wish me luck.

from clojure-maven-plugin.

OK, I found the problem. I'll do a PR as soon as I'm done.

from clojure-maven-plugin.

Thank you, Andrea.

#114

from clojure-maven-plugin.

@danielsz awesome! Will pull this in shortly ( probably tonight and release ).

from clojure-maven-plugin.

@talios Thank you! I love this project.

from clojure-maven-plugin.

Ahh - I see the commit message didn't mention the ticket so didn't auto-close. Manually doing so :)

from clojure-maven-plugin.

I updated the README to mention the new maven dependency/version - are all the arguments/system properties still valid?

from clojure-maven-plugin.

Yes, arguments/properties should be exactly as before, Thank you for updating the README with the new nREPL dependency/version. I forgot to do that myself. Sorry about that.

from clojure-maven-plugin.

...and released - should be in central within the hour.

from clojure-maven-plugin.

Awesome I was exactly trying to fetch it 😉

from clojure-maven-plugin.

@arichiardi if needed - you could probably resolve it from the oss.sonatype.org repositories directly - or just wait a bit longer :)

from clojure-maven-plugin.