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

Implement new metadata knobs as internal APIs #71449

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

eiriktsarpalis
Copy link
Member

@eiriktsarpalis eiriktsarpalis commented Jun 29, 2022

Performs internal refactorings in the metadata layer in anticipation of new APIs to be incorporated (cf. #63686 (comment), pending review of #71123).

Changes made

  • Adds ExtensionDataProperty, serialization callback, and Order knobs to the metadata as internal properties.
  • Improves XML documentation for the newly shipped APIs.
  • Ensures that the property sorting imlpementation is stable.
  • Fix an issue with nullability annotations in JsonPropertyDictionary.

Contributes to #63686.

@ghost
Copy link

ghost commented Jun 29, 2022

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

Performs internal refactorings in the metadata layer in anticipation of new APIs to be incorporated (cf. #63686 (comment)). Make the following changes:

  • Adds ExtensionDataProperty, serialization callback, and Order knobs to the metadata as internal properties.
  • Improves XML documentation for the newly shipped APIs.
  • Ensures that the property sorting imlpementation is stable.
  • Fix an issue with nullability annotations in JsonPropertyDictionary.

Contributes to #63686.

Author: eiriktsarpalis
Assignees: eiriktsarpalis
Labels:

area-System.Text.Json

Milestone: -

@eiriktsarpalis eiriktsarpalis merged commit cbed235 into dotnet:main Jun 30, 2022
@eiriktsarpalis eiriktsarpalis deleted the add-missing-metadata branch June 30, 2022 21:15
@ghost ghost locked as resolved and limited conversation to collaborators Jul 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants