cljfmt icon indicating copy to clipboard operation
cljfmt copied to clipboard

add deps.edn file

Open mpenet opened this issue 7 years ago • 4 comments
trafficstars

It's quite minimal but it works.

It can be useful to test non-released versions among other thing since we can pull dependencies via git directly.

I am not familiar with the cljs part of things, but I guess it shouldn't be too hard to add on top of that.

mpenet avatar Sep 14 '18 15:09 mpenet

I might add deps.edn in future, but for now I don't see the point of it if the package is already in Clojars. Testing unreleased versions is something that I'd rather not make easy, as I'd prefer people use the released builds.

weavejester avatar Sep 14 '18 16:09 weavejester

It's quite convenient, I waited a couple of weeks to get the fdef indentation fix (I know I could have done that via config too), that would have saved me some waiting.

But no big deal, I understand your position. Feel free to close this ticket if you'd like to do so.

mpenet avatar Sep 17 '18 09:09 mpenet

I'll keep this issue open a little while longer as I want to give this further consideration.

weavejester avatar Sep 17 '18 09:09 weavejester

I think having a deps.edn might be really nice. I want to pull in some recently merged changes (#193) but I don't know if they warrant a release.

djblue avatar May 18 '20 21:05 djblue

This can be closed now as there is a deps.edn:

 bb -Sdeps '{:deps {cljfmt/cljfmt {:git/url "https://github.com/weavejester/cljfmt" :git/sha "7dfd55d5dd0756f30311a90f206c2dd32e56d18b" :deps/root "cljfmt"}}}' -m cljfmt.main

borkdude avatar Feb 02 '23 15:02 borkdude