-
Notifications
You must be signed in to change notification settings - Fork 1.6k
markdig fixes - Part 3 #2554
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
markdig fixes - Part 3 #2554
Conversation
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.
LGTM. I've left one suggested nit unrelated to your PR and one comment. You can merge when you're ready, @mairaw.
@@ -596,7 +596,7 @@ | |||
<Docs> | |||
<param name="tasks">The array of tasks from which to continue.</param> | |||
<param name="continuationFunction">The function delegate to execute asynchronously when all tasks in the <paramref name="tasks" /> array have completed.</param> | |||
<param name="continuationOptions">One of the enumeration values that controls the behavior of the created continuation task. The NotOn* or OnlyOn* values are not valid.</param> |
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.
The strings NotOn* or OnlyOn*
and NotOn* and OnlyOn*
are found in a number of other overloads with a TaskContinuationOptions parameter. Do you want to change those as well?
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.
You mean in other types @rpetrusha?
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.
No. At least I thought that there were additional occurrences in this type.
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.
I just fixed those.
Co-Authored-By: Ron Petrusha <[email protected]>
No description provided.