test-refresh icon indicating copy to clipboard operation
test-refresh copied to clipboard

Launch nrepl server along with the test runner

Open vemv opened this issue 6 years ago • 2 comments

Hi!

Normally I connect Emacs (which might as well be any other editor/IDE) to an external nrepl process, for reliability: I simply lein repl in iTerm with a fixed port (as per profiles.clj).

The only problem being, a lein repl + a lein test-refresh is wasteful: that's 2 JVMs.

So I was thinking, what if lein test-refresh offers a :start-nrepl CLI option.

Does it sound like a feature your library could accept as a PR?

Cheers - Victor

vemv avatar Aug 19 '18 06:08 vemv

Thanks for asking. I'd accept a PR for this feature. This request has come up a couple times (maybe not on GitHub but in conversations at least).

I've been hesitant to add it because I don't think I'd enjoy being connected to a REPL where something else (in this case, lein-test-refresh) is in control of when the REPL's state changes.

jakemcc avatar Aug 19 '18 18:08 jakemcc

I've been hesitant to add it because I don't think I'd enjoy being connected to a REPL where something else (in this case, lein-test-refresh) is in control of when the REPL's state changes.

Good observation, the end result of my effort might be messier than I thought.

I'm trying a solution at Lein level first https://github.com/nrepl/lein-nrepl/pull/4 , as the problem seems generic. Might close this issue if that one works fine.

vemv avatar Aug 19 '18 23:08 vemv