This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-04-11
Channels
- # aleph (1)
- # architecture (18)
- # beginners (57)
- # boot (13)
- # cider (47)
- # cljs-dev (28)
- # cljsjs (13)
- # cljsrn (39)
- # clojure (258)
- # clojure-greece (16)
- # clojure-italy (2)
- # clojure-miami (1)
- # clojure-nl (9)
- # clojure-poland (15)
- # clojure-russia (369)
- # clojure-spec (53)
- # clojure-uk (49)
- # clojureremote (3)
- # clojurescript (135)
- # core-matrix (1)
- # cursive (11)
- # datascript (3)
- # datomic (4)
- # emacs (12)
- # figwheel (13)
- # hoplon (7)
- # interop (7)
- # jobs-discuss (33)
- # lumo (28)
- # off-topic (15)
- # om (7)
- # onyx (26)
- # pedestal (10)
- # perun (1)
- # planck (10)
- # re-frame (54)
- # reagent (16)
- # ring (5)
- # ring-swagger (50)
- # rum (2)
- # spacemacs (2)
- # unrepl (125)
- # untangled (12)
@dnolen why does this exist? https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/compiler.cljc#L567-L570
I was tracking down why multi-arity/variadic fn
did emit the @param {...*} var_args
jsdoc but defn
didn't
@dnolen just so I understand correctly: that variadic fn
emits a var_args
jsdoc even with :check-types :off
is the bug?
@thheller I don’t know, what do you think the problem is, and more importantly why does it matter? 🙂
I'm looking to enable :check-types
again, just going through each warning I encounter
I used {:check-types :warning}
which the above method doesn't recognise (expects :warn
which is incorrect), so variadic defn
never hard the proper var_args annotation
never really looked into the :check-types
things before but I have a goal in mind which I need it for
so it matters that {:check-types :warning}
isn't recognized in the comp/checking-types?
fn above
whatever it becomes it should be tied into spec and how it works should be considered details
yeah I'm getting that impression. Closure doesn't like the protocol dispatch it seems. WARNING - Property cljs$core$IHash$ never defined on o
WARNING - Function cljs.core.PersistentHashSet.createAsIfByAssoc: called with 2 argument(s). Function requires at least 1 argument(s) and no more than 1 argument(s).
WARNING - Function cljs.core.PersistentArrayMap.createAsIfByAssoc: called with 3 argument(s). Function requires at least 1 argument(s) and no more than 1 argument(s).
@thheller I agree it’s useful, but I just want it to be more idiomatic via spec integration
not intending to typecheck really, just looking if some extra type annotations could enable an idea I had for :infer-externs
for everyone out there suffering from the Closure Compiler async
bug, http://dev.clojure.org/jira/browse/CLJS-2006
they released a new version, v20170409