@dpsutton hey is there an JIRA ticket w/ your CLJS patch for the and/or work? I found the core.async one but no patch and it didn't link to anything
I think so
Should be attached. Let me look
link? Let me assign it myself and I can review and give some feedback
thanks!
linked in the async ticket now as well
great
I didn't have any prior art for how to turn one ast node into another. I thought i remembered stuff like this in tools analyzer cljs but that repo was marked deprecated.
yeah we might need to copy stuff from there or the analyzer for clj
if it turns out that this work needs a few AST helpers that we will reuse later that's fine and good
Another thing I had thought of, is it possible that closure might optimize this anyways and the ast transformation isn’t necessary?