puppetlabs / trapperkeeper

A services framework for Clojure / JVM applications.
https://tickets.puppetlabs.com/browse/TK
Apache License 2.0
587 stars 74 forks source link

(orch-2282) Updates trapperkeeper to use the latest nrepl/nrepl #280

Closed speedofdark closed 5 years ago

speedofdark commented 5 years ago

This is part of a series of changes to support nrepl/nrepl in trapperkeeper. It needs to be merged after the corresponding changes in clj-parent.

This change requires lein 2.9.0 or later to work; this is when the lein project changed from the legacy org.clojure/tools.nrepl to the supported nrepl/nrepl line of development.

This release drops support for JDK7, as nrepl/nrepl has a minimum requirement of JDK8.

Must not be merged until after https://github.com/puppetlabs/clj-parent/pull/167 is merged.

puppetcla commented 5 years ago

CLA signed by all contributors.

speedofdark commented 5 years ago

The fail in Travis is because its trying to run Java8 based nrepl/nrepl in a Java7 environment. I'm not sure if this matters?

speedofdark commented 5 years ago

I need to clarify some circular version references before we merge this. Resolved!