Fork me on GitHub
#cljdoc
<
2019-06-18
>
seancorfield02:06:27

@martinklepsch I happened to notice that one of my next.jdbc namespaces doesn't make it into the nav tree on the left (because it has no vars) https://cljdoc.org/d/seancorfield/next.jdbc/1.0.0/api/next.jdbc.connection -- I wondered if there was an option in cljdoc via metadata to force inclusion of a namespace?

martinklepsch09:06:16

The decision of not showing "empty" namespaces in the navigation was made ad-hoc and could be revisited. Do you have any thoughts on that general decision? @seancorfield

borkdude11:06:25

@seancorfield you can preview cljdoc docs on your own machine (e.g. via Docker): https://github.com/cljdoc/cljdoc/blob/master/doc/running-cljdoc-locally.md#running-locally-with-docker this prevented surprises before I deployed my libs a few times

seancorfield14:06:49

@borkdude yeah, no surprises. I was just curious about the decision.

👍 4
seancorfield14:06:42

@martinklepsch it's a perfectly reasonable default. A metadata option on the ns to override it would be nice. Not a big deal. I definitely think "empty" namespaces are an edge case.

borkdude14:06:21

I’m also curious why you would want a namespace without vars to be documented. what is the reason for its existence?

seancorfield16:06:08

@borkdude It has implementations of protocols so it's "important" from that point of view but the only docstring in it, I think, is on the ns itself.

seancorfield16:06:07

(it also contains a bunch of private vars and functions that support those implementations)

borkdude17:06:54

makes sense