This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-02-25
Channels
- # beginners (20)
- # boot (25)
- # cider (1)
- # cljs-dev (7)
- # cljsjs (1)
- # cljsrn (1)
- # clojure (79)
- # clojure-austin (2)
- # clojure-berlin (13)
- # clojure-dusseldorf (1)
- # clojure-germany (7)
- # clojure-russia (10)
- # clojure-serbia (1)
- # clojure-spec (18)
- # clojure-uk (4)
- # clojured (1)
- # clojurescript (90)
- # cursive (10)
- # datomic (7)
- # emacs (14)
- # hoplon (6)
- # luminus (16)
- # lumo (4)
- # numerical-computing (2)
- # om (25)
- # om-next (1)
- # onyx (11)
- # pedestal (10)
- # protorepl (1)
- # reagent (11)
- # remote-jobs (1)
- # ring (1)
- # rum (38)
- # spacemacs (5)
- # test-check (7)
- # untangled (122)
- # vim (1)
- # yada (8)
I just checked import
feature I was waiting for. it works in latest release! awesome, thanks guys!
@rmuslimov you mean ES6 import w/ Node.js resolution?
BOOM! the generated code uses async
name as variable/function name, but in ES6/ES2017 it has special meaning[1]
[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function
we will have to add it into js-reserved-arr
I guess
@dnolen I meant import like import {foo, bar} from './utils’;
, AFAIU node.js import resolution is not supported for now.
I’m experimenting with a fix, adding async into js-reserved
in analyzer.cljc
and js-reserved-arr
of core.cljs
creates another problem, it munges goog.async namespace as well, which is unwanted in this case, it breaks code like this:
https://github.com/clojure/core.async/blob/master/src/main/clojure/cljs/core/async/impl/dispatch.cljs#L29