Skip to content

Remove RequiresProcessIsolation from AsyncCovariantReturn#124302

Merged
MichalStrehovsky merged 1 commit intomainfrom
MichalStrehovsky-patch-1
Feb 12, 2026
Merged

Remove RequiresProcessIsolation from AsyncCovariantReturn#124302
MichalStrehovsky merged 1 commit intomainfrom
MichalStrehovsky-patch-1

Conversation

@MichalStrehovsky
Copy link
Member

We should probably just delete this test when tests for #124238 are added, but let's at least make it less expensive for now.

We should probably just delete this test when tests for #124238 are added, but let's at least make it less expensive for now.
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Feb 11, 2026
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 PR reduces the execution overhead of the AsyncCovariantReturn loader test by removing its process-isolation requirement, allowing it to run as part of the merged Loader test runner rather than as a standalone isolated process.

Changes:

  • Removed <RequiresProcessIsolation>true</RequiresProcessIsolation> from the AsyncCovariantReturn test project so it no longer forces OutputType=Exe / out-of-proc execution.

@MichalStrehovsky MichalStrehovsky merged commit 9cb0647 into main Feb 12, 2026
80 of 87 checks passed
@MichalStrehovsky MichalStrehovsky deleted the MichalStrehovsky-patch-1 branch February 12, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants