Skip to content

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.

@eiriktsarpalis eiriktsarpalis requested a review from krwq June 29, 2022 18:13
@ghost ghost assigned eiriktsarpalis Jun 29, 2022
@eiriktsarpalis eiriktsarpalis added this to the 7.0.0 milestone Jun 29, 2022
@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