babashka / neil

A CLI to add common aliases and features to deps.edn-based projects
MIT License
360 stars 26 forks source link

How about adding clj-reload in `:dev`? #231

Closed teodorlu closed 3 weeks ago

teodorlu commented 3 weeks ago

Please answer the following questions and leave the below in as part of your PR.


I've become accustomed to have tonsky/clj-reload available in the development profile. I missed clj-reload yesterday specifically to unload tests that no longer exist in the source files.

What do you think, is it OK to add clj-reload in local development?

borkdude commented 3 weeks ago

I'm fine with this only if you're not forced to use it. Is that the case?

teodorlu commented 3 weeks ago

Yes—This PR only puts the dep on the classpath, no need to use the :dev alias or use the dependency if you don't want to.

I don't think introducing a "forced way to reload Neil" is helpful; it's a small code base with no in-memory state, just being able to start a REPL and work is enough for me.