clj-commons / rewrite-clj

Rewrite Clojure code and edn
https://cljdoc.org/d/rewrite-clj
MIT License
592 stars 55 forks source link

ci: fix downloading deps for windows #253

Closed lread closed 9 months ago

lread commented 9 months ago

The download-deps bb task creates a basis for each deps.edn alias in-process to efficiently download all deps (work is done in build.clj).

Because the :build alias also included deps-deploy its many deps were brought in. This seems to somehow mess up bringing down conflicting(?) deps for other aliases, at least for Windows.

I've separated out deps-deploy to :deploy from :build alias to try to avoid the issue.

Also: switched to clojure.tools.build.api/create-basis, was using clojure.tools.deps/create-basis; no need to use deps ns when build api exposes this fn.

Also: was depending on stale org.clojure/tools.build maven dep, switched to current io.github.clojure/tools.build dep and updated to current version.

Closes #252