2023-09-11 cljdoc | Clojure Slack Archive

cljdoc

juhoteperi 2023-09-11T12:44:38.693439Z

Hey, has there been a recent change that could cause reitit main module analyze to fail? Seems to be due to language auto-detection and the fact that the main module doesn't contain any source files. I'm quite sure this used to work previously. https://cljdoc.org/builds/71954

✅ 1

opqdonut 2023-09-11T12:45:58.333589Z

I'm trying if specifying languages explicitly helps

opqdonut 2023-09-11T12:46:28.092669Z

https://github.com/metosin/reitit/pull/649

juhoteperi 2023-09-11T12:50:36.663409Z

Could be due to this change: https://github.com/cljdoc/cljdoc-analyzer/pull/83/files

opqdonut 2023-09-11T12:54:22.568119Z

I'm getting

2023-09-11 12:51:52,529 INFO  cljdoc-analyzer.runner - launching metagetta for: /home/runner/work/reitit/reitit/modules/reitit languages: :auto-detect
...
 Execution error (ExceptionInfo) at cljdoc-analyzer.metagetta.utils/infer-platforms-from-src-dir (utils.clj:118).
 no Clojure/Clojurescript sources found

even though I tried setting :cljdoc/languages in doc/cljdoc.edn and modules/reitit/doc/cljdoc.edn (see PR linked above)

lread 2023-09-11T13:46:19.689749Z

Thanks for reporting! I'll take a peek sometime soon.

lread 2023-09-11T13:50:57.002599Z

Oh. So the main reitit jar has no Clojure source.

unzip -l ~/Downloads/reitit-0.7.0-alpha5.jar
Archive:  /Users/lee/Downloads/reitit-0.7.0-alpha5.jar
  Length      Date    Time    Name
---------  ---------- -----   ----
      234  06-14-2023 16:10   META-INF/MANIFEST.MF
     9521  06-14-2023 16:10   META-INF/maven/metosin/reitit/pom.xml
     1313  06-14-2023 16:10   META-INF/leiningen/metosin/reitit/project.clj
        0  06-14-2023 16:10   META-INF/
        0  06-14-2023 16:10   META-INF/maven/
        0  06-14-2023 16:10   META-INF/maven/metosin/
        0  06-14-2023 16:10   META-INF/maven/metosin/reitit/
      105  06-14-2023 16:10   META-INF/maven/metosin/reitit/pom.properties
---------                     -------
    11173                     8 files

This used to be ok and still should be, but cljdoc now barfs on it.

lread 2023-09-11T15:24:27.385239Z

I'll take a peek sometime today.

lread 2023-09-11T20:48:27.017149Z

All fixed! Thanks for reporting!

opqdonut 2023-09-12T05:07:42.044389Z

confirmed: https://cljdoc.org/d/fi.metosin/reitit/0.7.0-alpha6/doc/introduction

opqdonut 2023-09-12T05:07:46.291459Z

thanks a ton!

👍 1

Clojurians Log v2

cljdoc