Skip to content

Commit 6dc3c10

Browse files
committed
print an info message that the :async feature is no longer needed
close #20
1 parent 87a27cd commit 6dc3c10

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

src/lib/devtools/async.cljs

+14-1
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,21 @@
11
(ns devtools.async
22
(:require-macros [devtools.oops :refer [oset ocall]])
3-
(:require [goog.async.nextTick :as next-tick]))
3+
(:require [goog.async.nextTick :as next-tick]
4+
[goog.labs.userAgent.browser :as ua]
5+
[devtools.context :as context]))
46

57
(defn ^:dynamic available? []
68
(exists? js/Promise))
79

10+
(def ^:dynamic fixed-chrome-version-for-async "65.0.3321")
11+
12+
(defn ^:dynamic needed? []
13+
(not (and (ua/isChrome) (ua/isVersionOrHigher fixed-chrome-version-for-async))))
14+
15+
(defn ^:dynamic get-not-needed-message []
16+
(str "cljs-devtools: the :async feature is no longer needed since Chrome " fixed-chrome-version-for-async ", "
17+
"see https://github.com/binaryage/cljs-devtools/issues/20"))
18+
819
(def ^:dynamic *installed* false)
920
(def ^:dynamic *original-set-immediate* nil)
1021

@@ -35,6 +46,8 @@
3546
(set! *installed* true)
3647
(oset js/Error ["stackTraceLimit"] js/Infinity)
3748
(install-async-set-immediate!)
49+
(when-not (needed?)
50+
(.info (context/get-console) (get-not-needed-message)))
3851
true))
3952

4053
(defn uninstall! []

0 commit comments

Comments
 (0)