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

Fix missing parameter descriptions in docs #212

Merged
merged 1 commit into from Sep 20, 2021

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Sep 20, 2021

The parameter descriptions were missing for any destructured parameters. Apparently TypeDoc doesn't support showing descriptions for destructured parameters (see here for more details).

I have forked TypeDoc to fix this problem, and published this fork as @gudahtt/typedoc@0.23.0. We can use this fork until this has been resolved upstream.

The parameter descriptions were missing for any destructured
parameters. Apparently TypeDoc doesn't support showing descriptions for
destructured parameters (see here for more details: [1]).

I have forked TypeDoc to fix this problem, and published this fork as
`@gudahtt/typedoc@0.23.0`. We can use this fork until this has been
resolved upstream.

[1]: TypeStrong/typedoc#1703
@Gudahtt Gudahtt requested a review from a team as a code owner September 20, 2021 18:07
Copy link
Contributor

@ryanml ryanml left a comment

Choose a reason for hiding this comment

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

nit: would it be more appropriate to fork this to the org? I'm fine with it either way

@Gudahtt
Copy link
Member Author

Gudahtt commented Sep 20, 2021

I could! I was hoping this would be a short-term thing. I can migrate it to the MetaMask org if I have trouble landing this upstream.

@ryanml
Copy link
Contributor

ryanml commented Sep 20, 2021

sounds good! lgtm

@Gudahtt Gudahtt merged commit 0027f50 into main Sep 20, 2021
@Gudahtt Gudahtt deleted the fix-typedoc-missing-parameter-descriptions branch September 20, 2021 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants