Skip to content

Generic metadata and type parameters fixup #2649

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

Merged
merged 18 commits into from
May 19, 2021

Conversation

jcollins-g
Copy link
Contributor

Fixes #2517. (Depending on whether or not we consider dart-lang/sdk#46064 blocking).

Rearranges how we handle typedefs to always render docs for typedefs as though they were declared in the new style, eliminating a lot of old special-casing.

Some sample renders:

(This one is missing metadata on the right hand side due to the analyzer issue)
Screen Shot 2021-05-18 at 3 18 03 PM

Legacy renders, all except NewGenericTypedef are using the old style of function typedefs:

Screen Shot 2021-05-18 at 3 20 06 PM

@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label May 18, 2021
@coveralls
Copy link

coveralls commented May 18, 2021

Coverage Status

Coverage increased (+0.2%) to 57.988% when pulling 0282676 on jcollins-g:minor-features-touchup into b8958f9 on dart-lang:master.

@jcollins-g jcollins-g requested a review from srawlins May 19, 2021 21:03
Copy link
Member

@srawlins srawlins left a comment

Choose a reason for hiding this comment

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

Very nice; implementation looks nice and consistent.

@jcollins-g jcollins-g merged commit 822504a into dart-lang:master May 19, 2021
@jcollins-g jcollins-g deleted the minor-features-touchup branch May 19, 2021 22:36
jcollins-g added a commit to jcollins-g/dartdoc that referenced this pull request May 20, 2021
jcollins-g added a commit that referenced this pull request May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow generic function types as type arguments
3 participants