noprompt / meander

Tools for transparent data transformation
MIT License
921 stars 55 forks source link

CI: exercise JVM variant, with clojurescript being present in the classpath #174

Closed vemv closed 3 years ago

vemv commented 3 years ago

See: https://github.com/noprompt/meander/pull/172#issuecomment-812314943

This way we ensure that the following:

(defmacro val-op
  {:private true}
  [x]
  (try
    (let [c (Class/forName "cljs.tagged_literals.JSValue")]
      `(.val ^"cljs.tagged_literals.JSValue" ~x))
    (catch ClassNotFoundException _
      `(.val ~x))))

...doesn't cause an issue in either branch of the try/catch.

noprompt commented 3 years ago

@vemv Thanks again! 👍