Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get docs working on cljdoc #380

Open
lread opened this issue Mar 24, 2022 · 1 comment
Open

Get docs working on cljdoc #380

lread opened this issue Mar 24, 2022 · 1 comment

Comments

@lread
Copy link

lread commented Mar 24, 2022

Problem

Docs for refactor-nrepl are failing to build on cljdoc, example:
https://cljdoc.org/d/refactor-nrepl/refactor-nrepl/3.5.2

(Since this repo links to cljdoc via a cljdoc badge, I assume that you'd prefer they work)

Steps already taken

@vemv has already asked cljdoc to only analyze for Clojure, this will help.

Other issues

Lein plugin

Cljdoc loads namespaces to discover the API.
Leiningen is not present and therefore analysis of the lein plugin will fail.
If I include leiningen-core as a provided dependency in refactor-nrepl's project.clj, cljdoc will include it and plugin analysis will succeed.
So that's good.

What public API should refactor-nrepl expose on cljdoc?

After adjusting for leningen, a local test exposes the following namespaces:

image

Which seems like a bit much, right?
I assume inlined-deps should not be documented.
What would be the API you would expect to be exposed on cljdoc?
Is the rest part of your supported public API?

Next steps

If you tell me what you'd like to see on cljdoc, I'm happy to follow up with a PR.
(and possibly changes to cljdoc itself, if appropriate).

@bbatsov
Copy link
Member

bbatsov commented Oct 26, 2022

(Since this repo links to cljdoc via a cljdoc badge, I assume that you'd prefer they work)

Yeah, for sure. For various reasons all of our projects have been kind of cursed and don't work with cljdoc (e.g. cider-nrepl and orchard don't work as well), which is a pity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants