search

burbma / cljs-cache

Port of clojure/core.cache
16 stars 7 forks source link

Clojars artifact depends on too many things #4

Open arichiardi opened 5 years ago

arichiardi commented 5 years ago

Hello!

I am trying this library but when I added it to my deps.edn I see a lot of things downloaded that probably should not be there (like boot namespaces):

Downloading: org/clojars/mmb90/cljs-cache/0.1.4/cljs-cache-0.1.4.pom from https://repo.clojars.org/
Downloading: org/clojure/core.async/0.2.385/core.async-0.2.385.pom from https://repo1.maven.org/maven2/
Downloading: adzerk/boot-test/1.1.2/boot-test-1.1.2.pom from https://repo.clojars.org/
Downloading: crisptrutski/boot-cljs-test/0.2.2-SNAPSHOT/boot-cljs-test-0.2.2-20160402.204547-3.pom from https://repo.clojars.org/
Downloading: org/clojars/mmb90/cljs-cache/0.1.4/cljs-cache-0.1.4.jar from https://repo.clojars.org/
Downloading: adzerk/boot-test/1.1.2/boot-test-1.1.2.jar from https://repo.clojars.org/
Downloading: crisptrutski/boot-cljs-test/0.2.2-SNAPSHOT/boot-cljs-test-0.2.2-20160402.204547-3.jar from https://repo.clojars.org/

Would you be open to a patch to review the dependencies and add :scope "test" where necessary?

arichiardi commented 5 years ago

Oh well there are actually many :scope "test" already in build.boot so I wonder where these things are coming from.