Skip to content
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

Update XsltArgumentList.xml #5121

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ericamick
Copy link

As written, the documentation currently suggests only one parameter per namespace is allowed. It should be one parameter with a given name per namespace.

Summary

Make clear that a duplicate name in the same namespace throws the exception.

Fixes #Issue_Number (if available)

As written, the documentation currently suggests only one parameter per namespace is allowed. It should be one parameter with a given name per namespace.
@opbld33
Copy link

opbld33 commented Nov 14, 2020

Docs Build status updates of commit 0eb0d49:

🕙 Pending: waiting for processors (8 builds ahead of you)

⚠️ Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@opbld33
Copy link

opbld33 commented Nov 15, 2020

Docs Build status updates of commit 0eb0d49:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Xml.Xsl/XsltArgumentList.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 requested a review from buyaa-n December 17, 2020 04:51
@@ -258,7 +258,7 @@

The <paramref name="name" /> is not a valid name according to the W3C XML specification.

The <paramref name="namespaceUri" /> already has a parameter associated with it.</exception>
The <paramref name="namespaceUri" /> already has a parameter called <paramref name="name" /> associated with it.</exception>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it meant it already has a parameter associated with the given name

Suggested change
The <paramref name="namespaceUri" /> already has a parameter called <paramref name="name" /> associated with it.</exception>
The <paramref name="namespaceUri" /> already has a parameter associated with the <paramref name="name" />.</exception>

Base automatically changed from master to main March 5, 2021 20:52
@gewarren gewarren requested review from joperezr and krwq as code owners March 5, 2021 20:52
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.

5 participants