This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2018-11-02
Channels
- # announcements (2)
- # aws (13)
- # beginners (52)
- # boot (10)
- # calva (2)
- # cider (23)
- # clara (23)
- # cljs-dev (16)
- # cljsrn (5)
- # clojure (69)
- # clojure-brasil (1)
- # clojure-conj (3)
- # clojure-dev (41)
- # clojure-india (2)
- # clojure-italy (39)
- # clojure-nl (5)
- # clojure-russia (2)
- # clojure-spec (5)
- # clojure-uk (51)
- # clojurescript (78)
- # code-reviews (13)
- # data-science (2)
- # datascript (22)
- # datomic (47)
- # duct (13)
- # emacs (4)
- # figwheel-main (45)
- # fulcro (85)
- # funcool (4)
- # jobs (9)
- # nrepl (106)
- # off-topic (5)
- # pathom (7)
- # pedestal (2)
- # re-frame (17)
- # reagent (32)
- # reitit (7)
- # ring-swagger (2)
- # shadow-cljs (33)
- # spacemacs (4)
- # specter (2)
- # tools-deps (203)
- # vim (1)
A potential (non-regression) defect surrounding externs inference:
$ clj -m cljs.main -co '{:infer-externs true}' -r
ClojureScript 1.10.339
cljs.user=> (defn foo? [x y] (< x (or y (.-MAX_VALUE js/Number))))
WARNING: cljs.core/<, all arguments must be numbers, got [nil #{nil js/Number}] instead at line 1 <cljs repl>
If you don’t specify :infer-externs
, then (.-MAX_VALUE js/Number)
is inferred as type js
and no warning is triggered. Alternatively, if you manually hint it with ^number
then the warning goes away.Hrm. I wonder if some npm
deps are currently botched. ClojureScript tests that previously passed, now fail. Here is an example of re-running the latest passing test for master in CI:
https://travis-ci.org/mfikes/clojurescript/builds/449630410
It is working again for me locally as well. (It was broken locally and in CI for perhaps an hour. Perhaps some glitch occurred that they cleared up quickly, so as to avoid another leftpad fiasco.)
I think the release notes for 1.10.439 are all in order. I’ll revise the date timestamp and put in the actual version number…
Sounds good. Here is the draft: https://github.com/clojure/clojurescript-site/blob/news-next/content/news/2018-11-02-release.adoc