Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Mar 10, 2022

In the description of the example for AmbiguousMatchException(String, Exception), the term 'class' or 'classes' is used when it should be 'method' or 'methods'. The proposed changed is to replace 'class' with 'method' and 'classes' with 'methods'

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

In the description of the example for AmbiguousMatchException(String, Exception), the term 'class' or 'classes' is used when it should be 'method' or 'methods'.  The proposed changed is to replace 'class' with 'method' and 'classes' with 'methods'
@ghost ghost self-requested a review as a code owner March 10, 2022 22:42
@opbld33
Copy link

opbld33 commented Mar 10, 2022

Docs Build status updates of commit de5bdfb:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
xml/System.Reflection/AmbiguousMatchException.xml ❌Error Details
❌Error Details

xml/System.Reflection/AmbiguousMatchException.xml

  • Line 0, Column 0: [Error: ECMA2Yaml_InternalError]
Intenal Several Error: System.Xml.XmlException: Data at the root level is invalid. Line 1, position 1.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.Throw(String res, String arg)
   at System.Xml.XmlTextReaderImpl.ParseRootLevelWhitespace()
   at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
   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)

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

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

Copy link
Contributor

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

@opbld34
Copy link

opbld34 commented Mar 10, 2022

Docs Build status updates of commit 67a5666:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Reflection/AmbiguousMatchException.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:

Copy link
Contributor

@buyaa-n buyaa-n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@buyaa-n buyaa-n merged commit a47340c into dotnet:main Mar 15, 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.

4 participants