cljdoc

https://cljdoc.org/ & https://github.com/cljdoc/cljdoc
martinklepsch 2019-02-28T18:27:36.001Z

@jayzawrotny whoops, just saw this again, sorry for the delay 😅

martinklepsch 2019-02-28T18:52:36.002100Z

@jayzawrotny it looks like you're building docs for a leiningen template, which is not really supported very well

2019-02-28T19:01:31.003400Z

Ah ok, thought that may be the case and as I thought about it the less sense it made to generate docs at all. The README is probably just fine, thanks for looking into it.

dominicm 2019-02-28T19:05:57.004Z

I suppose a list of files or directories to ignore would solve this?

dominicm 2019-02-28T19:06:01.004200Z

Oh actually

dominicm 2019-02-28T19:06:12.004600Z

I might have a better fix

dominicm 2019-02-28T19:07:44.004900Z

hmm, you're not using tools.namespace directly?

2019-02-28T19:08:34.005200Z

Nope

dominicm 2019-02-28T19:12:08.005500Z

Sorry, that was more directed at martin & cljdoc

martinklepsch 2019-02-28T19:13:12.006500Z

at dinner now will reply later, we use codox under the hood which may use tools namespace, not 100% sure

dominicm 2019-02-28T19:14:44.007100Z

:cljdoc.api/namespaces could one use this to perform the appropriate filtering perhaps? currently it's restricted to just the hardcoded things though 🙂

dominicm 2019-02-28T19:14:50.007300Z

I can't see how namespaces are found normally

dominicm 2019-02-28T19:14:57.007500Z

maybe analysis is the wrong place

dominicm 2019-02-28T19:15:17.007700Z

now I know where to look though 🙂

dominicm 2019-02-28T19:16:52.008900Z

https://github.com/weavejester/codox/blob/59cafeffcadc37d0eae0973011977e3916ddadbb/codox/src/codox/reader/clojure.clj#L7 looks like it does use it yeah, okay. So you're basically running into TNS-45. There is a JUXT fork of tns which applies the patches there, but it requires a bit of annoying work (globally excluding tools.namespace). Doing that would fix the problem though. You could probably monkey patch tns from cljdoc if you wanted.

dominicm 2019-02-28T19:17:02.009300Z

https://dev.clojure.org/jira/browse/TNS-45 tns-45 is the bane of my existence 🙂

dominicm 2019-02-28T19:18:41.010Z

I have considered going on a crusade like nrepl did with a forked version of tns which fixes this. I think tns has more users though, so might be harder, and it doesn't seem to bother that many people?

2019-02-28T21:00:25.010400Z

oh! My mistake sorry