pitch-io/cljest

Fix `cljdoc` build

Closed this issue · 1 comments

jo-sm commented

At the moment it seems to fail, see https://cljdoc.org/builds/67878. Would ^:no-doc to the right functions fix the issue?

jo-sm commented

Turns out, no, ^:no-doc didn't fix it, the issue wasn't the analysis of the function but that cljest.core was getting the user defined formatters namespace in a variable, which depends on the config, and cljdoc-analyzer doesn't have a cljest.edn file, eventually causing the analysis to throw and fail (see Slack thread). Lee Read helped figure this out, shoutout to him!

Additionally, some of the library dependencies are optional and only needed if you're going to use the helpers (reagent, re-frame), so some of the helper namespaces also didn't analyze and require some optional dependency workaround.