Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing render/deps.edn in config.clj file when using git deps #349

Closed
pfeodrippe opened this issue Dec 16, 2022 · 2 comments · Fixed by #350
Closed

Missing render/deps.edn in config.clj file when using git deps #349

pfeodrippe opened this issue Dec 16, 2022 · 2 comments · Fixed by #350

Comments

@pfeodrippe
Copy link
Contributor

pfeodrippe commented Dec 16, 2022

With the following deps.edn

{:deps
 {org.clojure/clojure {:mvn/version "1.11.1"}
  io.github.nextjournal/clerk {:git/sha "23a2c898735af7e02738a2974cfa1e4e1fdd170d"}}

 :paths ["src"]}

and clj file

(ns abc
  (:require
   [nextjournal.clerk :as clerk]))

I'm having this issue

1. Caused by java.lang.Exception
   Visiting /Users/paulo.feodrippe/dev/abcd/resources failed

                   fs.cljc:  192  babashka.fs$walk_file_tree$fn__5740/invoke
                   fs.cljc:  207  babashka.fs$walk_file_tree$reify__5742/visitFileFailed
                Files.java: 2813  java.nio.file.Files/walkFileTree
                   fs.cljc:  193  babashka.fs$walk_file_tree/invokeStatic
                   fs.cljc:  174  babashka.fs$walk_file_tree/invoke
                   fs.cljc:  286  babashka.fs$match/invokeStatic
                   fs.cljc:  246  babashka.fs$match/invoke
                   fs.cljc:  335  babashka.fs$glob/invokeStatic
                   fs.cljc:  312  babashka.fs$glob/invoke
                   fs.cljc:  328  babashka.fs$glob/invokeStatic
                   fs.cljc:  312  babashka.fs$glob/invoke
                dejavu.clj:   95  nextjournal.dejavu/glob-sources
                dejavu.clj:   94  nextjournal.dejavu/glob-sources
                dejavu.clj:  111  nextjournal.dejavu/cljs-files/fn
                  core.clj: 2770  clojure.core/map/fn
              LazySeq.java:   42  clojure.lang.LazySeq/sval
              LazySeq.java:   51  clojure.lang.LazySeq/seq
                   RT.java:  535  clojure.lang.RT/seq
                  core.clj:  139  clojure.core/seq

It was complaining about a missing render/deps.edn in another run.

@mk mk closed this as completed in #350 Dec 16, 2022
mk added a commit that referenced this issue Dec 16, 2022
@mk
Copy link
Member

mk commented Dec 16, 2022

@pfeodrippe thanks for the report, great to have regressions be caught this quickly. Let me know if the attached patch doesn't solve the issue.

@pfeodrippe
Copy link
Contributor Author

pfeodrippe commented Dec 17, 2022

@pfeodrippe thanks for the report, great to have regressions be caught this quickly. Let me know if the attached patch doesn't solve the issue.

@mk Yes, it's working, thanks!! =D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants