Skip to content

Conversation

@Evangelink
Copy link
Member

Removed older .NET runtimes from the configuration.

Removed older .NET runtimes from the configuration.
@Evangelink Evangelink requested review from Youssef1313 and Copilot and removed request for Youssef1313 November 26, 2025 15:47
@Evangelink Evangelink enabled auto-merge (squash) November 26, 2025 15:47
Copilot finished reviewing on behalf of Evangelink November 26, 2025 15:49
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 updates the global.json configuration to remove outdated .NET runtimes (3.1.32, 6.0.36, and 7.0.20) which are no longer in support and not actively used in the codebase.

  • Removed .NET 3.1.32, 6.0.36, and 7.0.20 from the dotnet runtime section
  • Removed .NET 3.1.32 and 6.0.36 from the dotnet/x86 runtime section
  • Kept .NET 8.0.18 and 9.0.7 in the dotnet section, while other sections only retain 9.0.7
Comments suppressed due to low confidence (1)

global.json:14

  • The aspnetcore section is missing the 8.0.18 runtime while the dotnet section includes it. Since the repository has an Aspire test project (samples/public/DemoMSTestSdk/ProjectUsingAspire) that targets net8.0 and uses Aspire.Hosting.Testing which requires ASP.NET Core runtime, consider whether 8.0.18 should also be included in the aspnetcore array for consistency:
"aspnetcore": [
  "8.0.18",
  "9.0.7"
]
      "aspnetcore": [
        "9.0.7"
      ]

@Evangelink Evangelink merged commit 458db16 into main Nov 27, 2025
8 checks passed
@Evangelink Evangelink deleted the Evangelink-patch-1 branch November 27, 2025 10:35
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.

3 participants