Skip to content

Fix operator docs #8692

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

Merged
merged 9 commits into from
Jan 25, 2023
Merged

Conversation

jairosoto05
Copy link
Contributor

Reverse operator docs and change the word implicitly to explicitly Fixes #8318

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

Reverse operator docs and change the word implicitly to explicitly
Fixes dotnet#8318
@jairosoto05 jairosoto05 requested a review from a team as a code owner December 6, 2022 18:27
@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Dec 6, 2022
@dnfadmin
Copy link

dnfadmin commented Dec 6, 2022

CLA assistant check
All CLA requirements met.

@ghost ghost added the area-System.Text.Json label Dec 6, 2022
@ghost
Copy link

ghost commented Dec 6, 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

Reverse operator docs and change the word implicitly to explicitly Fixes #8318

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

Author: jairosoto05
Assignees: -
Labels:

area-System.Text.Json, community-contribution

Milestone: -

@opbld34
Copy link

opbld34 commented Dec 6, 2022

Learn Build status updates of commit d21aa67:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details
xml/System.Text.Json.Nodes/JsonNode.xml ❌Error Details

  • Line 0, Column 0: [Error: ECMA2Yaml_File_LoadFailed] Failed to load 1 files, aborting...

xml/System.Text.Json.Nodes/JsonNode.xml

  • Line 0, Column 0: [Error: ECMA2Yaml_InternalError]
Intenal Several Error: System.Xml.XmlException: Name cannot begin with the '.' character, hexadecimal value 0x2E. Line 551, position 142.
   at System.Xml.XmlTextReaderImpl.ParseQName(Boolean isQName, Int32 startOffset, Int32& colonPos)
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.

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.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@ghost ghost added the needs-author-action An issue or pull request that requires more info or actions from the author. label Dec 6, 2022
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@ghost ghost removed the needs-author-action An issue or pull request that requires more info or actions from the author. label Dec 6, 2022
@opbld31
Copy link

opbld31 commented Dec 6, 2022

Learn Build status updates of commit fbf2ea8:

✅ 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:

@jairosoto05 jairosoto05 requested a review from gewarren December 6, 2022 19:55
jairosoto05 and others added 6 commits December 6, 2022 18:15
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@opbld30
Copy link

opbld30 commented Dec 6, 2022

Learn Build status updates of commit 1bb665a:

✅ 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:

@opbld30
Copy link

opbld30 commented Dec 6, 2022

Learn Build status updates of commit 6f26ae9:

✅ 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:

@opbld31
Copy link

opbld31 commented Dec 7, 2022

Learn Build status updates of commit ded2f7d:

✅ 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:

@jairosoto05 jairosoto05 requested a review from gewarren December 7, 2022 02:15
@eiriktsarpalis eiriktsarpalis merged commit 9b07a10 into dotnet:main Jan 25, 2023
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.

JsonNode explicit operator docs reversed
7 participants