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(graindoc): Use value_descriptions and type_declarations defined by the module signature #1241

Merged
merged 2 commits into from
May 22, 2022

Conversation

ospencer
Copy link
Member

We previously did a lookup on the types in the module's signature, but that type is the original type rather than what the module is trying to expose. This change uses what's defined in the signature.

@ospencer ospencer requested a review from a team May 21, 2022 22:55
@ospencer ospencer self-assigned this May 21, 2022
@ospencer ospencer force-pushed the oscar/fix-graindoc-type-reporting branch 2 times, most recently from c96a69f to 829c344 Compare May 21, 2022 23:27
@ospencer ospencer force-pushed the oscar/fix-graindoc-type-reporting branch from 829c344 to a5b31ee Compare May 21, 2022 23:28
Copy link
Member

@phated phated left a comment

Choose a reason for hiding this comment

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

Amazing! Thank you so much for digging into that for me 🙇 I love that we can show types like Map<k, v> now!

Copy link
Member

@marcusroberts marcusroberts left a comment

Choose a reason for hiding this comment

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

Excellent!

@ospencer ospencer merged commit 5896242 into main May 22, 2022
@ospencer ospencer deleted the oscar/fix-graindoc-type-reporting branch May 22, 2022 22:42
@github-actions github-actions bot mentioned this pull request May 31, 2022
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.

3 participants