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

Fixed typo in remarks section of OnCompleted #62982

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

slask
Copy link
Contributor

@slask slask commented Dec 18, 2021

Fix #62248

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Dec 18, 2021
@ghost
Copy link

ghost commented Dec 18, 2021

Tagging subscribers to this area: @dotnet/area-system-runtime-compilerservices
See info in area-owners.md if you want to be subscribed.

Issue Details

Fix #62248

Author: slask
Assignees: -
Labels:

area-System.Runtime.CompilerServices

Milestone: -

@slask
Copy link
Contributor Author

slask commented Dec 19, 2021

Hi @buyaa-n . Looks like the pipeline fails for reasons unrelated to the change. Could be a transient issue? Can the pipeline be retried?

@buyaa-n
Copy link
Member

buyaa-n commented Dec 20, 2021

Looks like the pipeline fails for reasons unrelated to the change. Could be a transient issue? Can the pipeline be retried?

Yes, it is unrelated and doesn't look like needs further attention, thanks!

@buyaa-n buyaa-n merged commit 192db36 into dotnet:main Dec 20, 2021
@@ -78,7 +78,7 @@ internal TaskAwaiter(Task task)
/// <param name="continuation">The action to invoke when the await operation completes.</param>
/// <exception cref="System.ArgumentNullException">The <paramref name="continuation"/> argument is null (Nothing in Visual Basic).</exception>
/// <exception cref="System.InvalidOperationException">The awaiter was not properly initialized.</exception>
/// <remarks>This method is intended for compiler user rather than use directly in code.</remarks>
/// <remarks>This method is intended for compiler use rather than use directly in code.</remarks>
Copy link
Member

Choose a reason for hiding this comment

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

This only changed this one comment. The same typo appears in other places, e.g. in the very next function on L91.

@ghost ghost locked as resolved and limited conversation to collaborators Feb 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Runtime.CompilerServices community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Possible typo: "This method is intended for compiler user..."
3 participants