This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2021-06-02
Channels
- # announcements (1)
- # asami (89)
- # aws (10)
- # babashka (41)
- # beginners (71)
- # calva (25)
- # cider (3)
- # clj-kondo (65)
- # cljdoc (15)
- # cljs-dev (3)
- # cljsrn (8)
- # clojure (56)
- # clojure-europe (44)
- # clojure-italy (1)
- # clojure-nl (2)
- # clojure-uk (48)
- # clojured (7)
- # clojurescript (17)
- # conjure (6)
- # cursive (3)
- # datomic (9)
- # emacs (11)
- # events (3)
- # fulcro (3)
- # helix (3)
- # honeysql (7)
- # hugsql (6)
- # introduce-yourself (2)
- # jobs (2)
- # kaocha (4)
- # luminus (1)
- # nrepl (2)
- # off-topic (10)
- # pathom (7)
- # philosophy (3)
- # polylith (27)
- # reagent (18)
- # reitit (3)
- # remote-jobs (7)
- # reveal (3)
- # shadow-cljs (9)
- # slack-help (5)
- # tools-deps (9)
- # vim (48)
I wonder if we should try to add cljdoc-analyzer (or just its cljs metagetta tests) to https://github.com/cljs-oss/canary
Could be a win/win. Cljdoc team would discover problems early due to ClojureScript changes, and ClojureScript team would know early if their changes have affected cljdoc.
that sounds like a good idea
Because I don’t know/remember: why does cljdoc-analyzer supports parsing source with unknown tagged literals? I see from the https://github.com/cljdoc/cljdoc-analyzer/issues/32?.
yeah, that #js tag was present in a cljc file I believe and broke Clojure analysis
I wonder if this data_readers.clj is fulcro’s way of addressing? https://github.com/fulcrologic/fulcro/blob/develop/src/main/data_readers.clj
@U0522TWDA I think you worked on this one, do you remember?
It will be problem for any CLJC lib that is using #js and, in general, for any lib using any custom data literals. But the fix I made fixes that for all possible cases?
Yes, if you had data readers like those referenced above at root of the classpath then it would work. IMHO it's good to ignore unknown literals because 99% they are irrelevant for those parts of code Cljdoc care about, ie function names and docstrings. No?
There is a lib with custom literals for Java time classes. Should we fail parsing any lib using those?
Thanks for the reply! I only ask because the latest version of ClojureScript broke cljdoc-analyzer’s metagetta in this area. And I was wondering if we really need to support unknown tagged literals. Or should we just expect projects to include appropriate data_readers.clj/cljc? (Maybe this would not be enough? I’m a bit fuzzy on what might be required).