Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 21, 2025

This PR removes the general-testing NuGet package source from the root NuGet.config file. This feed was previously needed for a prototype Roslyn compiler for the runtime-async effort, but is no longer required.

Changes

  • Removed the general-testing package source entry
  • Removed the associated comment explaining its purpose

The remaining package sources (dotnet-public, dotnet-tools, dotnet-eng, dotnet-libraries, dotnet-libraries-transport, dotnet10, dotnet10-transport, dotnet11, and dotnet11-transport) are still configured and functional.

Validation

  • Confirmed the XML is well-formed
  • Verified no other files in the repository reference the general-testing feed
Original prompt

Remove the following line from NuGet.config in the root of the repository:

This will remove the 'general-testing' package source from the package sources list. No other package sources should be changed. Commit message: "Remove general-testing feed from NuGet.config".

This pull request was created as a result of the following prompt from Copilot chat.

Remove the following line from NuGet.config in the root of the repository:

This will remove the 'general-testing' package source from the package sources list. No other package sources should be changed. Commit message: "Remove general-testing feed from NuGet.config".


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@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 Oct 21, 2025
Copilot AI changed the title [WIP] Remove general-testing package source from NuGet.config Remove general-testing feed from NuGet.config Oct 21, 2025
Copilot AI requested a review from agocke October 21, 2025 23:08
Copilot finished work on behalf of agocke October 21, 2025 23:08
@am11 am11 added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 22, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@akoeplinger akoeplinger marked this pull request as ready for review October 22, 2025 15:26
@Copilot Copilot AI review requested due to automatic review settings October 22, 2025 15:26
@akoeplinger akoeplinger merged commit c0fbb06 into main Oct 22, 2025
156 checks passed
@akoeplinger akoeplinger deleted the copilot/remove-general-testing-feed branch October 22, 2025 15:26
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 removes the obsolete general-testing NuGet package source from the root NuGet.config file. The feed was previously required for a prototype Roslyn compiler related to the runtime-async effort but is no longer needed.

Key changes:

  • Removed the general-testing package source entry and its explanatory comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants