-
Notifications
You must be signed in to change notification settings - Fork 6k
Compare async/await to ContinueWith #47075
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
base: main
Are you sure you want to change the base?
Conversation
Hi @copilot Can you start working on this again? |
Sure! I'm resuming work on this issue. I'll be adding a comparison between ContinueWith and async/await approaches to show how async/await simplifies asynchronous code, especially when chaining multiple operations. |
Co-authored-by: BillWagner <[email protected]>
Co-authored-by: BillWagner <[email protected]>
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.
Overall, this is good. I had a few comments to address before it's final.
...ynchronous-programming/snippets/index/ContinueWith-comparison/ContinueWith-comparison.csproj
Outdated
Show resolved
Hide resolved
...us-programming/snippets/index/AsyncBreakfast-ContinueWith/AsyncBreakfast-ContinueWith.csproj
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,139 @@ | |||
using System; |
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.
Delete this file because the snippets aren't included in the article.
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.
Deleted the unused AsyncBreakfast-ContinueWith/Program.cs file in commit 3285a72.
… misleading text, use tag-based snippets Co-authored-by: BillWagner <[email protected]>
Addressed all the review comments in commit 3285a72. |
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.
Much better. This is ready for final review.
Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.
Original issue description:
Fixes #21296.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
Internal previews