Skip to content

Conversation

akoeplinger
Copy link
Member

It's not needed there and will raise a NuGet package pruning warning

It's not needed there and will raise a NuGet package pruning warning
@Copilot Copilot AI review requested due to automatic review settings August 13, 2025 16:25
Copy link
Contributor

@Copilot 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 refines the conditional reference to System.ValueTuple package to only target .NET Framework projects, removing it from .NET Standard 2.0 where it's not needed and causes NuGet package pruning warnings.

  • Updated conditional reference logic for System.ValueTuple package
  • Changed from excluding .NET Core App to explicitly including only .NET Framework

Reviewed Changes

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

File Description
Microsoft.AspNetCore.InternalTesting.csproj Modified System.ValueTuple reference condition to target only .NET Framework
Microsoft.Extensions.Logging.AzureAppServices.csproj Modified System.ValueTuple reference condition to target only .NET Framework

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@github-actions github-actions bot added the needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically label Aug 13, 2025
@akoeplinger akoeplinger enabled auto-merge (squash) August 13, 2025 16:26
@akoeplinger akoeplinger merged commit ab9e263 into dotnet:main Aug 13, 2025
30 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-rc1 milestone Aug 13, 2025
@akoeplinger akoeplinger deleted the valuetuple branch August 13, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-area-label Used by the dotnet-issue-labeler to label those issues which couldn't be triaged automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants