Skip to content

Conversation

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM @gewarren

Let's :shipit:

@gewarren gewarren merged commit 4f20ce9 into dotnet:main Apr 29, 2025
22 checks passed
@gewarren gewarren deleted the warnings-april25 branch April 29, 2025 15:25
Copilot AI added a commit that referenced this pull request Nov 4, 2025
The note incorrectly stated that TargetFrameworks property is ignored if TargetFramework is specified. The actual behavior is the opposite - when both properties are present, TargetFrameworks (plural) takes precedence and TargetFramework (singular) is ignored. Updated the note to correctly reflect this behavior. Fixes #45942 Co-authored-by: gewarren <24882762+gewarren@users.noreply.github.com>
gewarren added a commit that referenced this pull request Nov 4, 2025
) * Initial plan * Fix TargetFrameworks documentation note The note incorrectly stated that TargetFrameworks property is ignored if TargetFramework is specified. The actual behavior is the opposite - when both properties are present, TargetFrameworks (plural) takes precedence and TargetFramework (singular) is ignored. Updated the note to correctly reflect this behavior. Fixes #45942 Co-authored-by: gewarren <24882762+gewarren@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gewarren <24882762+gewarren@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment