Skip to content

Conversation

@gewarren
Copy link
Contributor

  • Remove internal only remark
  • Add see also link to new conceptual doc

@gewarren gewarren requested a review from a team as a code owner September 23, 2022 19:30
@ghost ghost assigned gewarren Sep 23, 2022
@ghost
Copy link

ghost commented Sep 23, 2022

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

Issue Details
  • Remove internal only remark
  • Add see also link to new conceptual doc
Author: gewarren
Assignees: -
Labels:

area-System.Text.Json

Milestone: -

@opbld34

This comment was marked as outdated.

@gewarren gewarren closed this Sep 23, 2022
@gewarren gewarren reopened this Sep 23, 2022
@opbld32

This comment was marked as outdated.

@gewarren
Copy link
Contributor Author

Close/reopen for build.

@gewarren gewarren closed this Sep 23, 2022
@gewarren gewarren reopened this Sep 23, 2022
@opbld31

This comment was marked as outdated.

@opbld33

This comment was marked as outdated.

@opbld33
Copy link

opbld33 commented Sep 23, 2022

Docs Build status updates of commit 3aca760:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Text.Json.Serialization.Metadata/JsonCollectionInfoValues`1.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonObjectInfoValues`1.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonParameterInfoValues.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonPropertyInfo.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonTypeInfo.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld31

This comment was marked as outdated.

Copy link
Member

@eiriktsarpalis eiriktsarpalis 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!

@gewarren gewarren enabled auto-merge September 26, 2022 17:08
@opbld32
Copy link

opbld32 commented Sep 26, 2022

Docs Build status updates of commit 5bfff12:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Text.Json.Serialization.Metadata/JsonObjectInfoValues`1.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonParameterInfoValues.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonPropertyInfo.xml ✅Succeeded View
xml/System.Text.Json.Serialization.Metadata/JsonTypeInfo.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@gewarren gewarren merged commit 082c04f into dotnet:main Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants