-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Added a basic example #583
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
Added a more practical example
updated the description to make it clear it will get run on the same thread
Resolved all comments
Added an example of a wrong use case and an example of a unit test using it
@BillWagner could you review the changes now? included a wrong use case and a unit test as you mentioned |
Added the initial paragraph back
I've been remiss in commenting on this PR; I apologize for that. The delay is that I'm struggling with the right recommendation to update it. In general, we try to write positive guidance rather than negative guidance. There's two reasons:
I'm checking with the team to get recommendations. |
After a lot of debate, I'm closing this. The API reference is the wrong location for conceptual guidance, which is what this PR does. That means developers find this only when they've already reach the point where they are searching for a specific method. A better way to provide this guidance is to create an article in our conceptual section that provides the correct way to ensure that a GUI remains responsive during long-running tasks. Consider this type of guidance as part fo dotnet/docs#6373 or dotnet/docs#951 |
Summary
Added a basic example of the usage
Fixes dotnet/docs#6272