leonoel / cloroutine

Coroutine support for clojure
Eclipse Public License 2.0
228 stars 10 forks source link

Warnings when loading missionary.core #23

Closed slifin closed 1 year ago

slifin commented 2 years ago
Screenshot 2022-04-10 at 22 06 39

deps

Dependency graph on a fairly minimal app that uses Missionary + Clojure 1.11.1 ^

In recent versions of Clojure they've introduced a few new functions such as update-vals these were historically already defined in clojure.tools.analyzer

If we look at Clojure tools analyzer today they've fixed those warnings for the new version of Clojure: https://github.com/clojure/tools.analyzer/blob/master/src/main/clojure/clojure/tools/analyzer/passes/uniquify.clj (by explicitly excluding those functions which prevents the warning)

but the version of Clojure tools analyzer we're depending on in this project (indirectly via org.clojure/tools.analyzer.jvm) does not: https://github.com/clojure/tools.analyzer/blob/20a6aac20045c6d9bb82ccbb40f3c86ca839f4e3/src/main/clojure/clojure/tools/analyzer/passes/uniquify.clj

Is it possible to bump org.clojure/tools.analyzer.jvm dependency up?

leonoel commented 1 year ago

Fixed in release 11