Skip to content

Conversation

@jcollins-g
Copy link
Contributor

This fixes an issue blocking https://dart-review.googlesource.com/c/sdk/+/185903. Dartdoc was still trying to cast a type where it shouldn't. Also add some test coverage as besides building the SDK this case was uncovered.

@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label Jul 1, 2021
@jcollins-g jcollins-g requested a review from scheglov July 1, 2021 17:23
Copy link
Contributor

@scheglov scheglov left a comment

Choose a reason for hiding this comment

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

Thank you :-)

@jcollins-g jcollins-g merged commit c9621b9 into dart-lang:master Jul 1, 2021
dart-bot pushed a commit to dart-lang/sdk that referenced this pull request Jul 1, 2021
This is not a full release of dartdoc; this patch
(matching dart-lang/dartdoc#2714),
helps unblock https://dart-review.googlesource.com/c/sdk/+/185903.

Change-Id: I6bfea10d253b251a61525459d0fb1c26649929a1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/205700
Reviewed-by: Konstantin Shcheglov <[email protected]>
Commit-Queue: Janice Collins <[email protected]>
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.

2 participants