Skip to content

Conversation

@JimDaly
Copy link
Contributor

@JimDaly JimDaly commented Apr 19, 2022

Summary

Describe your changes here.

Per comments in #7941 , removing remarks that say 'This method is intended for compiler use rather than for use in application code.'

At this point, the right thing for TaskAwaiter and TaskAwaiter is probably to just strike those sentence from the docs, though some extra verbiage around the distinction from Task.Wait would be welcome

@JimDaly JimDaly requested a review from kouvel as a code owner April 19, 2022 17:49
@ghost
Copy link

ghost commented Apr 19, 2022

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

Issue Details

Summary

Describe your changes here.

Per comments in #7941 , removing remarks that say 'This method is intended for compiler use rather than for use in application code.'

At this point, the right thing for TaskAwaiter and TaskAwaiter is probably to just strike those sentence from the docs, though some extra verbiage around the distinction from Task.Wait would be welcome

Author: JimDaly
Assignees: -
Labels:

area-System.Threading.Tasks

Milestone: -

@opbld33
Copy link

opbld33 commented Apr 19, 2022

Docs Build status updates of commit 3f8aac4:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Threading.Tasks/Task`1.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
Copy link
Contributor

Closing in favor of #7968.

@gewarren gewarren closed this Apr 19, 2022
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.

3 participants