Skip to content

Use WaitForExitAsync from polyfill#7460

Merged
Youssef1313 merged 3 commits intomicrosoft:mainfrom
SimonCropp:use-WaitForExitAsync-from-polyfill
Feb 25, 2026
Merged

Use WaitForExitAsync from polyfill#7460
Youssef1313 merged 3 commits intomicrosoft:mainfrom
SimonCropp:use-WaitForExitAsync-from-polyfill

Conversation

@SimonCropp
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 24, 2026 09:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request removes conditional compilation for .NET Standard 2.0 by leveraging the Polyfill package to provide WaitForExitAsync for all target frameworks. The change simplifies the codebase by eliminating the platform-specific workaround that was needed for .NET Standard 2.0.

Changes:

  • Removed #if !NETSTANDARD2_0 and #else conditional compilation blocks that handled Process.WaitForExitAsync differently for .NET Standard 2.0
  • Unified the code path to use WaitForExitAsync with cancellation token across all target frameworks

…ocessExtensions.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 24, 2026 10:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copy link
Member

@nohwnd nohwnd left a comment

Choose a reason for hiding this comment

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

need one more look

@Youssef1313 Youssef1313 merged commit 642bcb8 into microsoft:main Feb 25, 2026
13 of 14 checks passed
@SimonCropp SimonCropp deleted the use-WaitForExitAsync-from-polyfill branch February 25, 2026 12:18
Evangelink pushed a commit that referenced this pull request Mar 2, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants