Skip to content
This repository has been archived by the owner on Jul 31, 2021. It is now read-only.

Open top-level type documentation for methods. #55

Closed
wants to merge 1 commit into from
Closed

Conversation

enocom
Copy link
Contributor

@enocom enocom commented Oct 30, 2019

Fixes #53.

@enocom enocom requested review from tvanhens and a team October 30, 2019 04:26
@enocom
Copy link
Contributor Author

enocom commented Oct 30, 2019

@tvanhens I could do some more work to open a URL with the correct fragment, but have stopped here to make sure the approach makes sense.

Copy link

@aesterline aesterline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

(with-meta (symbol name)
{:doc (render-docs docs)
:arglists (list (vec (cons 'this (mapv (comp symbol :name) parameters))))})
(with-meta
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The metadata should go on the symbol above (which will get put on the var that gets bound) rather than the function.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll make the change.

@enocom
Copy link
Contributor Author

enocom commented Oct 30, 2019

Some upcoming work from @tvanhens will conflict with this change. I'm going to re-open this once that other work lands.

@enocom enocom closed this Oct 30, 2019
@enocom enocom deleted the browse-fix branch November 28, 2019 17:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cdk/browse on static methods doesn't work
3 participants