Skip to content

Switch test runner to global.json#11096

Merged
dipeshmsft merged 2 commits into
mainfrom
dev/ygerges/test-runner-globaljson
Oct 26, 2025
Merged

Switch test runner to global.json#11096
dipeshmsft merged 2 commits into
mainfrom
dev/ygerges/test-runner-globaljson

Conversation

@Youssef1313

@Youssef1313 Youssef1313 commented Sep 8, 2025

Copy link
Copy Markdown
Member

If dotnet/sdk#50673 gets merged, the insertion of SDK with that change will require the change in this PR.

FYI @dipeshmsft

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings September 8, 2025 15:03
@Youssef1313 Youssef1313 requested a review from a team as a code owner September 8, 2025 15:03
@Youssef1313 Youssef1313 marked this pull request as draft September 8, 2025 15:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the test runner configuration by moving it from the legacy dotnet.config file to the global.json file format, aligning with changes in the .NET SDK.

  • Removes the test runner configuration from dotnet.config
  • Adds test runner configuration to global.json under a new "test" section

Reviewed Changes

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

File Description
global.json Adds new "test" section with Microsoft.Testing.Platform runner configuration
dotnet.config Removes legacy test runner configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dotnet-policy-service dotnet-policy-service Bot added PR metadata: Label to tag PRs, to facilitate with triage draft labels Sep 8, 2025
@Youssef1313 Youssef1313 marked this pull request as ready for review October 14, 2025 20:51
@Youssef1313

Copy link
Copy Markdown
Member Author

@dipeshmsft Please review.

@dipeshmsft dipeshmsft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@dipeshmsft dipeshmsft merged commit 2a554cd into main Oct 26, 2025
8 checks passed
@dipeshmsft dipeshmsft deleted the dev/ygerges/test-runner-globaljson branch October 26, 2025 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants