juxt / clip

Light structure and support for dependency injection
MIT License
228 stars 15 forks source link

no tools.namespace in deps.edn - juxt.clip.repl fails #20

Closed frap closed 3 years ago

frap commented 3 years ago

trying out 1st option (aka repl) on startrek get juxt.clip.repl error in that tools/namespace/repl isnt available. Fixed it in startrek by adding org.clojure/tools.namespace {:mvn/version "1.1.0"} to startrek deps - but it should be a dependency of clip no?

SevereOverfl0w commented 3 years ago

Sorry that this tripped you up. Its optional dependency of clip, you need to add it yourself if you're using that namespace.

This is documented under https://github.com/juxt/clip#reloaded

Clip provides a namespace for easily setting up a reloaded workflow. You will need to add a dependency on tools.namespace to your project.

Closing as I can't think of a way to make this clearer, but if you can, please let me know!

frap commented 3 years ago

Ok

Sorry didnt read far enough down on clip.

Regards A

Gas @.*** Home: PP6R+7P +64 21 629488 If there is no track, you can’t be off track.

On Sat, 15 May 2021 at 12:28, Dominic Monroe @.***> wrote:

Sorry that this tripped you up. Its optional dependency of clip, you need to add it yourself if you're using that namespace.

This is documented under https://github.com/juxt/clip#reloaded

Clip provides a namespace for easily setting up a reloaded workflow. You will need to add a dependency on tools.namespace to your project.

Closing as I can't think of a way to make this clearer, but if you can, please let me know!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/juxt/clip/issues/20#issuecomment-841570562, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJGBCIUDZCCC4EFUUW76OTTNW5Z3ANCNFSM44Z77UHA .