Setting default TargetPlatformVersion

[sfoslund]

Fixes dotnet/sdk#11233

cc @wli3

[vatsan-madhavan]

Does this affect anything besides CsWinRT scenarios ?

[wli3]

This change is more about future net5.0, net6.0 work. Detail is in there dotnet/sdk#11233

[vatsan-madhavan]

I’m asked after having looked there. The description there is a bit obscure but I found a link to dotnet/sdk#11556, which seemed suggestive.

[wli3]

@vatsan-madhavan Daniel commented.

@vatsan-madhavan @ryalanms, is there anything else you need to move forward?

[vatsan-madhavan]

I’m all good - thanks for follow up and explaining! LGTM!

[dsplaisted]

My opinion is that we should put this type of Windows-specific logic (as well as the logic to reference CSWinRT targeting packs, etc.) in dotnet/sdk, and the targets in dotnet/wpf should be the ones that are specifically for WPF.

I don't want to hold anything up if this way allows us to make progress faster, but that's where I'd like to end up.

@wli3 @sfoslund @vatsan-madhavan

[wli3]

Lets' keep this going. We don't have much time left. We could do the shuffling in RC or post 5.0

[sfoslund]

@vatsan-madhavan @ryalanms is there anything else that needs to be done here?

[ryalanms]

Lets' keep this going. We don't have much time left. We could do the shuffling in RC or post 5.0

@wli3: Can you make sure to track this refactoring work in another issue? Thanks.

[sfoslund]

I don't have merge access, but this should be ready to go. Thanks!