-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Added documentation for new Code Coverage in C# Fixes #7635 #7664
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
Conversation
|
@microsoft-github-policy-service agree |
|
Thanks @RonakRahane for contributing. Looping in @webreidi @katiesavage to review this PR. |
webreidi
left a comment
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 few updates for clarification and style.
ntrogh
left a comment
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.
@RonakRahane I revised the text to align with our writing style. Appreciate your contribution.
ntrogh
left a comment
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.
@RonakRahane LGTM!
Fixes #7635