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
improve article quality #20775
improve article quality #20775
Conversation
First, remove redundant and irrelevant links. Then, do a general edit pass on the article. Fixes AB#1763499
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Syntax highlighting
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - just a couple of nit suggestions.
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Outdated
Show resolved
Hide resolved
docs/csharp/programming-guide/classes-and-structs/named-and-optional-arguments.md
Show resolved
Hide resolved
…tional-arguments.md Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
These snippets include Office interop.
I'm overriding the CI build issues (for now) because it's an unrelated sample in the same folder, and it's because Office is not installed on our CI build system. In time, the right solution is to move or archive the (very small number) of articles that rely on office interop. |
First, remove redundant and irrelevant links.
Then, do a general edit pass on the article.
Fixes AB#1763499