tolitius / mount

managing Clojure and ClojureScript app state since (reset)
Eclipse Public License 1.0
1.22k stars 88 forks source link

(mount.core/start (mount.core/only #{})) should be a no-op #65

Closed zane closed 7 years ago

zane commented 7 years ago

Versions

[mount "0.1.10"]
[org.clojure/clojure "1.9.0-alpha14"]

Steps to reproduce the issue

(mount.core/defstate do-not-start-me
  :start (println "Darn. I started."))

(mount.core/start (mount/only #{}))

Expected result

Nothing is printed.

Actual result

Darn. I started. is printed.

tolitius commented 7 years ago

good catch. the fix is in 0.1.11-SNAPSHOT.

dev=> (defstate do-not-start-me 
                :start (println "Darn. I started."))
#'dev/do-not-start-me
dev=>

dev=> (mount/start (mount/only #{}))
{:started #{}}

Let me know if it works for you.