Skip to content
This repository has been archived by the owner on Mar 18, 2022. It is now read-only.

Commit

Permalink
Add deps.edn
Browse files Browse the repository at this point in the history
  • Loading branch information
@rynkowsg
rynkowsg committed Mar 16, 2022
1 parent 0359914 commit 5d703e6
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions deps.edn
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
;; this file is added only to develop Babashka scripts
;; in typical Clojure environment I use: Cursive + regular nREPL
{:paths ["src"]
:deps {org.clojure/clojure {:mvn/version "1.10.2"}
metosin/malli {:mvn/version "0.8.4"}}}

3 comments on commit 5d703e6

@piotr-yuxuan
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm very interested in porting malli-cli on babashka! The reason why I haven't moved from Leiningen to deps.edn completely is because the automation tooling I use is a bit drafty as of now, and only focuses on deps.edn.

That being said, if there is anything I may do to ease the port, just let me know. Very enthusiastic about reviewing any PR, commenting any issues, and discussing ideas about how to make it happen.

@rynkowsg
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @piotr-yuxuan,

Re babashka,
I initally made a babashka script, but when added started adding CLI support to it with malli-cli I converted it to just deps.edn based script. I must say, I did not made an investigation, just briefly looked through the problem and it looked that malli itself doesn't seem to work out of the box with babashka. At least not the malli imported with babashka.deps/add-deps. I don't remember the issue. I found a pod-babashka-malli, but then well, assuming malli-cli requires malli, I assumed it would not work. That is why I just moved to deps.edn for this case. At the end the boot start does not matter for me at this stage, and launching Clojure via deps.edn is good enough.

Re adding deps.edn
to be honest I added deps.edn on top of the change in my fork only to feed my script's deps.edn with a fixed malli-cli, via deps tools support for using git, e.g.

{
...
:deps {...
        rynkowsg/malli-cli  {:git/url "https://github.com/rynkowsg/malli-cli.git" :sha "5d703e68047f76c602fa97e49c4587a5e1d9e5a5"}
        ...
}}

Once the change is merged, I'll simply upgrade the version and dump the fork, including this commit :)

@piotr-yuxuan
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow-yp here: piotr-yuxuan#23

Please sign in to comment.