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

Process typeDetails information from symbol graphs. #529

Merged
merged 2 commits into from
Mar 31, 2023

Conversation

pdwilson12
Copy link
Contributor

@pdwilson12 pdwilson12 commented Mar 31, 2023

Bug/issue #, if applicable: rdar://107432025

Summary

Converts new typeDetails information in symbol graph into base-type details and allowed type declarations for RenderProperty entities, such as dictionary keys and HTTP parameters. This data is rendered on the page as part of the key or parameter documentation as a "Possible types" attribute, listing the individual types.

Dependencies

Requires changes on type_details branch of pdwilson12/swift-docc-symbolkit tracked in swiftlang/swift-docc-symbolkit#56

Testing

Build docc and preview the DictionaryData.docc fixture bundle.

Steps:

  1. Run docc preview on the DictionaryData.docc fixture
  2. View the Artist page and confirm "Possible types" entry of monthOfBirth key.
  3. Modify the symbol graph for monthOfBirth key, adding/modifying the typeDetails entry and verify changes reflected on page.

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests
  • Ran the ./bin/test script and it succeeded
  • Updated documentation if necessary

Convert data into base-type details and allowed type declarations for `RenderProperty` entities, such as dictionary keys and HTTP parameters.
Copy link
Contributor

@QuietMisdreavus QuietMisdreavus left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@pdwilson12
Copy link
Contributor Author

@swift-ci please test

@pdwilson12 pdwilson12 merged commit f7cc54a into swiftlang:main Mar 31, 2023
@pdwilson12 pdwilson12 deleted the type_details branch March 31, 2023 23:48
pdwilson12 added a commit to pdwilson12/swift-docc that referenced this pull request Mar 31, 2023
* Process `typeDetails` information from symbol graphs.

Convert data into base-type details and allowed type declarations for `RenderProperty` entities, such as dictionary keys and HTTP parameters.

* Update SymbolKit dependency to pick up latest commit.
pdwilson12 added a commit that referenced this pull request Apr 1, 2023
* Process `typeDetails` information from symbol graphs. (#529)

* Process `typeDetails` information from symbol graphs.

Convert data into base-type details and allowed type declarations for `RenderProperty` entities, such as dictionary keys and HTTP parameters.

* Update SymbolKit dependency to pick up latest commit.

* Update SymbolKit dependency to point to release/5.9 branch.
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.

2 participants