Skip to content

Conversation

arviedelgado
Copy link
Contributor

Docs is updated to align with the Source.

Summary

The Docs says:
Summary: Gets the root JsonNode. If the current JsonNode is a root, null is returned.
Remarks: The current node is returned if it is a root.

The Source says:
Summary: Gets the root JsonNode.
Remarks: The current node is returned if it is a root.

@arviedelgado arviedelgado requested a review from a team as a code owner November 26, 2022 03:26
@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.Text.Json labels Nov 26, 2022
@ghost
Copy link

ghost commented Nov 26, 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

Docs is updated to align with the Source.

Summary

The Docs says:
Summary: Gets the root JsonNode. If the current JsonNode is a root, null is returned.
Remarks: The current node is returned if it is a root.

The Source says:
Summary: Gets the root JsonNode.
Remarks: The current node is returned if it is a root.

Author: arviedelgado
Assignees: -
Labels:

area-System.Text.Json, community-contribution

Milestone: -

@opbld31

This comment was marked as outdated.

@BillWagner
Copy link
Member

closing and reopening for a fresh build.

@BillWagner BillWagner closed this Nov 28, 2022
@BillWagner BillWagner reopened this Nov 28, 2022
@opbld30
Copy link

opbld30 commented Nov 28, 2022

Learn Build status updates of commit fed0461:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Text.Json.Nodes/JsonNode.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 124d993 into dotnet:main Dec 1, 2022
@gewarren
Copy link
Contributor

gewarren commented Dec 1, 2022

Thanks @arviedelgado

@arviedelgado arviedelgado deleted the patch-1 branch December 6, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Text.Json community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants