Skip to content

Port System.Text.Json source comments to Docs #2360

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.

Sign up for GitHub

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

Closed
wants to merge 8 commits into from
Closed

Port System.Text.Json source comments to Docs #2360

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
f7e0281
Port System.Text.Json source comments to Docs
carlossanlop Apr 24, 2019
278fb4e
Apply suggestions from code review
carlossanlop Apr 25, 2019
de476f8
Adding missing remarks and some more exceptions.
carlossanlop Apr 25, 2019
a3f5ee4
Adding missing remarks and some more exceptions.
carlossanlop Apr 25, 2019
2d2075f
Update JsonElement.xml
carlossanlop Apr 25, 2019
5517bd6
Update Utf8JsonWriter.xml
carlossanlop Apr 26, 2019
12b1158
Apply suggestions from code review
May 7, 2019
1068a9b
Apply suggestions from code review
carlossanlop May 7, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions xml/System.Text.Json/JsonDocument.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,8 @@
<param name="readerOptions">Options to control the reader behavior during parsing.</param>
<summary>Parse a sequence as UTF-8-encoded text representing a single JSON value into a JsonDocument.</summary>
<returns>A JsonDocument representation of the JSON value.</returns>
<remarks>To be added.</remarks>
<remarks>The <see cref="T:System.Buffers.ReadOnlySequence`1" /> may be used for the entire lifetime of the JsonDocument object, and the caller must ensure that the data therein does not change during the object lifetime.
Because the input is considered to be text, a UTF-8 Byte-Order-Mark (BOM) must not be present.</remarks>
<exception cref="T:System.Text.Json.JsonReaderException"><paramref name="utf8Json" /> does not represent a valid single JSON value.</exception>
<exception cref="T:System.ArgumentException"><paramref name="readerOptions" /> contains unsupported options.</exception>
</Docs>
Expand Down Expand Up @@ -125,7 +126,8 @@
<param name="readerOptions">Options to control the reader behavior during parsing.</param>
<summary>Parse memory as UTF-8-encoded text representing a single JSON value into a JsonDocument.</summary>
<returns>A JsonDocument representation of the JSON value.</returns>
<remarks>To be added.</remarks>
<remarks>The <see cref="T:System.ReadOnlyMemory`1" /> value will be used for the entire lifetime of the JsonDocument object, and the caller must ensure that the data therein does not change during the object lifetime.
Because the input is considered to be text, a UTF-8 Byte-Order-Mark (BOM) must not be present.</remarks>
<exception cref="T:System.Text.Json.JsonReaderException"><paramref name="utf8Json" /> does not represent a valid single JSON value.</exception>
<exception cref="T:System.ArgumentException"><paramref name="readerOptions" /> contains unsupported options.</exception>
</Docs>
Expand Down Expand Up @@ -153,7 +155,7 @@
<param name="readerOptions">Options to control the reader behavior during parsing.</param>
<summary>Parse text representing a single JSON value into a JsonDocument.</summary>
<returns>A JsonDocument representation of the JSON value.</returns>
<remarks>To be added.</remarks>
<remarks>The <see cref="T:System.ReadOnlyMemory`1" /> value may be used for the entire lifetime of the JsonDocument object, and the caller must ensure that the data therein does not change during the object lifetime.</remarks>
<exception cref="T:System.Text.Json.JsonReaderException"><paramref name="json" /> does not represent a valid single JSON value.</exception>
<exception cref="T:System.ArgumentException"><paramref name="readerOptions" /> contains unsupported options.</exception>
</Docs>
Expand Down
2 changes: 1 addition & 1 deletion xml/System.Text.Json/JsonElement+ObjectEnumerator.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@
<Docs>
<summary>Returns an enumerator that iterates the properties of an object.</summary>
<returns>An <see cref="T:System.Text.Json.JsonElement.ObjectEnumerator" /> value that can be used to iterate through the object.</returns>
<remarks>To be added.</remarks>
<remarks>The enumerator will enumerate the properties in the order they are declared, and when an object has multiple definitions of a single property they will all individually be returned (each in the order they appear in the content).</remarks>
</Docs>
</Member>
<Member MemberName="MoveNext">
Expand Down
Loading