Regression of #3

[vemv]

Hey there again,

I just tried upgrading [expound "0.7.2"] -> [expound "0.8.3"].

But my cljs tests started failing with that, with the same exact error that the #3 OP attaches: Cannot read property 'cljs$lang$applyTo' of undefined.

I could fix it by adding #?(:cljs [goog.string.format]) in my codebase, just like 131d86c does.

However, the issue itself is a regression, thought it'd be worth informing of it to you.

Can attach a repro if needed but this one seems straightforward.

Cheers - V

[bhb]

Thanks for reporting this! It's unfortunate the tests didn't catch this mistake. It looks like I broke this in 22655c1

[bhb]

@vemv Out of curiosity, are you testing on an older version of CLJS? I am just curious because I can repro with

clojure -Sdeps '{:deps {org.clojure/clojurescript {:mvn/version "1.10.439"} expound {:mvn/version "0.8.3"}}}' -m cljs.main -re node -e "(require '[expound.alpha :as expound]) (expound.alpha/expound int? 5.6)"

but not

clojure -Sdeps '{:deps {org.clojure/clojurescript {:mvn/version "1.10.514"} expound {:mvn/version "0.8.3"}}}' -m cljs.main -re node -e "(require '[expound.alpha :as expound]) (expound.alpha/expound int? 5.6)"

(or any version newer than 1.10.514)

Regardless, this is still a bug I will fix.

[vemv]

I'm using org.clojure/clojurescript "1.10.597"

Thanks for the fix!

[bhb]

@vemv Thanks again for reporting this. Fixed in 0.8.4