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.

Sign up for GitHub

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

Jump to bottom

Why is otel-default-providers_ docstring not showing in cljdoc? #36

Closed
ptaoussanis opened this issue Nov 28, 2024 · 2 comments
Closed

Why is otel-default-providers_ docstring not showing in cljdoc? #36

ptaoussanis opened this issue Nov 28, 2024 · 2 comments
Assignees
@ptaoussanis
Labels
bug Something isn't working docs Documentation related input welcome Looking for feedback or ideas
Milestone
v1 final

Comments

@ptaoussanis
Copy link
Member

ptaoussanis commented Nov 28, 2024 •
edited
Loading

It's not obvious to me why the rendered docstring isn't showing.

Need to investigate. Code is here.
@ptaoussanis ptaoussanis added bug? Possible bug input welcome Looking for feedback or ideas docs Documentation related labels Nov 28, 2024
@ptaoussanis ptaoussanis self-assigned this Dec 20, 2024
@ptaoussanis ptaoussanis added this to the v1 final milestone Dec 20, 2024
@ptaoussanis
Copy link
Member Author

Hmm, seems to be related to the remote declarations.

Seems these might be picked up first by the cljdoc evaluation, and somehow causing confusion.

Will see if I can find a workaround.

@ptaoussanis ptaoussanis added bug Something isn't working and removed bug? Possible bug labels Dec 20, 2024
ptaoussanis added a commit that referenced this issue Dec 20, 2024
@ptaoussanis
[fix] [#36] Fix missing cljdoc docstrings
df02c9b 
These remote declarations were unnecessary (vestigial), and seemed
to be causing issues with cljdoc's analysis.
@ptaoussanis
Copy link
Member Author

Appears to be resolved by removing the (anyway unnecessary) remote declarations 👍

ptaoussanis added a commit that referenced this issue Dec 20, 2024
@ptaoussanis
[fix] [#36] Fix missing cljdoc docstrings
b58ec73 
These remote declarations were unnecessary (vestigial), and seemed
to be causing issues with cljdoc's analysis.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ptaoussanis ptaoussanis

Labels
bug Something isn't working docs Documentation related input welcome Looking for feedback or ideas
Projects
None yet
Milestone
v1 final
Development

No branches or pull requests
1 participant
@ptaoussanis