Fork me on GitHub
#cljs-dev
<
2018-07-05
>
Mikko Harju08:07:50

What would be the best resource to get acquainted with the CLJS compiler API? I’d be interested in reading in CLJS files for doing some static analysis purposes (looking up e.g. re-frame subs and building dependency graphs etc) I just blindly tried the analyze-file but it needs all the dependencies to be fed to it and it was not completely obvious as how to actually do that. Any good starting points?

bronsa09:07:43

arggh sorry @mfikes, I told jenkins to cut a tools.reader release after TRDR-52 but didn't notice it failed to push it

bronsa09:07:46

releasing 1.3.0 now

👍 4
mfikes12:07:41

Canary to the rescue again 🙂 It caught https://dev.clojure.org/jira/browse/CLJS-2807

dnolen12:07:47

@mikko looking at some of the analyzer tests is probably an OK starting point

dnolen12:07:12

another good way in the past was to work on a small issue that needs fixing

dnolen12:07:42

@ambrosebs do you want to take 2807? we should probably address that first before the others

Mikko Harju13:07:36

@dnolen Yeah – exactly where I started off a while ago. Will ask more when I bump into issues – thanks!

dnolen16:07:54

@mikko you might want to play around with analyze-form-seq or whatever that helper is called

dnolen16:07:02

also there’s a var for disabling analyzing deps

ambrosebs17:07:04

@dnolen CLJS-2807 is ready to go

darwin17:07:54

@ambrosebs I just gave you write access to the canary repo, if you are having troubles running canary locally, you can commit into “jobs” branch of that repo to trigger a new canary build. in your case something like job --compiler-repo --compiler-rev e03e416b972848200c18f23dcad2f918a3832932, see https://github.com/cljs-oss/canary/tree/jobs

ambrosebs17:07:36

@darwin I don't seem to have permission to push to cljs-oss/canary?

darwin17:07:16

in github ui I still see “Awaiting frenchy64's response”, I think you have to accept the invitation firts

darwin17:07:53

@ambrosebs btw. in --compiler-rev you can use branch/tag names or other forms git checkout understands, so --compiler-rev CLJS-2807 should work as well

darwin18:07:09

@ambrosebs your patch worked, there are some failed projects in canary job#466, but for a different reason - they do not respect custom --compiler-repo, I reported it here https://github.com/cljs-oss/canary/issues/5, I think @mfikes fixes it eventually

mfikes18:07:53

👍:skin-tone-2: