Fork me on GitHub
#cljs-dev
<
2018-07-03
>
ambrosebs14:07:55

@dnolen next patch ready to screen (add :quote op). It's more invasive than I expected, I did my best to minimize the changes. https://dev.clojure.org/jira/browse/CLJS-2801

dnolen14:07:04

thanks will take look

mfikes15:07:31

Cool! I was hoping that was the case. 🙂

kommen15:07:10

ah, I just now see your comment there

kommen15:07:58

hope to find some time to give the slate example a run with the latest closure

ambrosebs17:07:13

trying to use cljs.analyzer.api/resolve with a fresh repl... how do I do that?

user=> (require '[cljs.analyzer.api :as api] '[cljs.env :as env] '[cljs.analyzer :as ana])
nil
user=> (env/ensure (api/in-cljs-user (ana/load-core) (api/resolve (api/empty-env) 'cljs.core/map)))
nil

ambrosebs17:07:28

a fresh lein repl fwiw

dnolen19:07:22

@ambrosebs load-core doesn’t do what you think it does

dnolen19:07:37

it should be re-named load-core-macros or something

ambrosebs20:07:06

ah 🙂

dnolen20:07:59

setting up an environment where you can use resolve is not that simple (was intended for macro writers, who don’t need to bother with that)

dnolen20:07:14

see cljs.analyzer-tests for more guidance

ambrosebs20:07:14

ah of course.

dnolen20:07:18

pretty sure at a minimum you need to

dnolen20:07:29

1) bind cljs.analyzer/*cljs-ns*

dnolen20:07:40

2) wrap in cljs.env/with-compiler-env

dnolen20:07:22

3) wrap in cljs.compiler/with-core-cljs