Skip to content
Permalink
Browse files

REPL should make no assumptions about the namespaces loaded by the ru…

…ntime

otherwise reloading the browser doesn't allow requiring a namespace
since it thinks its already loaded
  • Loading branch information
thheller committed Sep 17, 2018
1 parent fbac46a commit f6694aaa5459591556a5e83f939885b70924d3b0
Showing with 1 addition and 13 deletions.
  1. +1 −13 src/main/shadow/cljs/repl.clj
@@ -177,20 +177,8 @@
{:keys [reload-deps] :as new-ns-info}
(ns-form/merge-repl-require ns-info require-form)

known-deps
(-> #{}
(into (:deps ns-info))
;; :deps already had the string aliased
;; need to remember which strings we aliased
;; so we don't reimport the whole thing again
(into (keys (:js-deps ns-info)))
;; (require 'the.thing :reload)
(util/reduce-> disj reload-deps))

new-deps
(->> (:deps new-ns-info)
(remove known-deps)
(into []))
(:deps new-ns-info)

[new-sources state]
(res/resolve-repl state (:name new-ns-info) new-deps)

0 comments on commit f6694aa

Please sign in to comment.
You can’t perform that action at this time.