juxt / clip

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

juxt.clip.core/ref no longer expanded in 0.22.0 #21

Closed apa512 closed 3 years ago

apa512 commented 3 years ago

After updating from 0.21.0 to 0.22.0, juxt.clip.core/ref is no longer replaced by the started component.

config.edn:

{:components
 {:a {:start 1}
  :b {:start (prn ">>>" (juxt.clip.core/ref :a) (type (juxt.clip.core/ref :a))}}

Code:

(clip.core/start (aero.core/read-config (clojure.java.io/resource "config.edn")))

In 0.22.0:

">>>" (clip/ref :a) clojure.lang.PersistentList

In 0.21.0:

">>>" 1 java.lang.Long

Am I correct in thinking that 0.21.1 is the expected behavior? Or have I been doing something weird that just happened to work in earlier versions?

SevereOverfl0w commented 3 years ago

Broken in 5b344d94aff56c2f57a8f7088af0c212e047ea14

SevereOverfl0w commented 3 years ago

Fixed in 0.23.0

apa512 commented 3 years ago

Thank you