Skip to content

Address feedback on RequiresProcessIsolation guidelines in tests#125035

Merged
MichalStrehovsky merged 2 commits intoMichalStrehovsky-patch-1from
copilot/sub-pr-125034
Mar 2, 2026
Merged

Address feedback on RequiresProcessIsolation guidelines in tests#125035
MichalStrehovsky merged 2 commits intoMichalStrehovsky-patch-1from
copilot/sub-pr-125034

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 2, 2026

Addresses reviewer feedback on #125034 regarding the RequiresProcessIsolation documentation.

Changes

  • requiresprocessisolation.md: Folds the standalone AutoreleasePoolSupport rule into the RuntimeHostConfigurationOption rule, since AutoreleasePoolSupport is just an MSBuild shorthand that expands into a RuntimeHostConfigurationOption item. References Microsoft.NET.ILLink.targets as the source of truth for other such shorthand properties. Renumbers all subsequent rules and fixes a pre-existing duplicate rule-17 numbering error.

  • testing.md: Adds a forward link from the existing "When to make a test RequiresProcessIsolation" section to requiresprocessisolation.md.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

… add testing.md link

Co-authored-by: MichalStrehovsky <13110571+MichalStrehovsky@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP on RequiresProcessIsolation guidelines in tests Address feedback on RequiresProcessIsolation guidelines in tests Mar 2, 2026
@MichalStrehovsky MichalStrehovsky marked this pull request as ready for review March 2, 2026 06:32
@MichalStrehovsky MichalStrehovsky merged commit 9edd16d into MichalStrehovsky-patch-1 Mar 2, 2026
16 checks passed
@MichalStrehovsky MichalStrehovsky deleted the copilot/sub-pr-125034 branch March 2, 2026 06:32
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants