Fork me on GitHub
#cljdoc
<
2023-02-18
>
Sam Ritchie16:02:18

@lee okay, another clerk-utils issue… https://app.circleci.com/pipelines/github/cljdoc/builder/40103/workflows/1a2aa813-9bef-494b-b3c0-33ec5aa86186/jobs/56477 the analyzer is looking for the user namespace… but I can’t see why it is doing that

Sam Ritchie16:02:31

oh… maybe this is what @U04V70XH6 warned me against? is cljdoc parsing the clj files in my template ?

lread16:02:39

Thanks for reporting @U017QJZ9M7W! I'll take a peek sometime today.

Sam Ritchie16:02:14

sounds good! I’ve also gone ahead and changed all of the clj and cljs extensions to tmpl, as @U04V70XH6 suggested in #C019ZQSPYG6. that hopefully will get us set on the next release

Sam Ritchie16:02:43

maybe something is strange though, since I don’t know if my hooks namespaces in resources get analyzed? https://github.com/mentat-collective/Clerk-Utils/blob/main/resources/clj-kondo.exports/org.mentat/clerk-utils/hooks/mentat/clerk_utils/viewers.clj like this will fail for sure, given my declared dependencies…

Sam Ritchie17:02:10

I guess I can :no-doc the hooks namespaces? is that required?

lread17:02:26

Yeah, namespaces with :no-doc are not explicitly loaded during analysis (but note, they can be implicitly loaded, by your other namespaces, probably not the case for you here though). :no-doc a good way to tell cljdoc (and codox for that matter) that a namespace should be excluded from your public API.

Sam Ritchie17:02:59

Okay, that should cover it… and the tmpl change hopefully will fix the template getting pulled in

lread17:02:23

Lemme me know if there is anything else to look into here, happy to help if there are still mysteries.

seancorfield00:02:14

The hooks can have a .clj-kondo extension as I recall. Check next.jdbc (in at a cat show right now and can't check)

Sam Ritchie15:02:28

I took that approach, thanks all!

lread16:02:22

Sweet! Btw, I have not gotten a response from the author of flexmark-java yet about those ref links in images yet.... so they'll continue to render badly for the near term.

Sam Ritchie16:02:50

no problem, thanks for checking in on that