From e308ca96fc85e6639086810d0bbaaabbd06ddb23 Mon Sep 17 00:00:00 2001
From: Eriawan Kusumawardhono <eriawan@live.com>
Date: Sat, 9 Oct 2021 02:11:57 +0700
Subject: [PATCH] Fixes #787 About unnecessary ItemGroup with condition in
 Splat.csproj (#788)

* Fixes #787
Remove ItemGroup that has conditon with TFM that starts with `net46`in Splat.csproj file.
Having this condition of TFM net46* is unnecessary, Because defined TargetFrameworks does not contain NET Fx related TFM.

* remove the net46 and net472 as well because these TFMs are in the Splat.Drawing as suggested in issue comment:
 https://github.com/reactiveui/splat/issues/787#issuecomment-938583942
---
 src/Splat/Splat.csproj | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/Splat/Splat.csproj b/src/Splat/Splat.csproj
index ac428f5c9..529d46acb 100644
--- a/src/Splat/Splat.csproj
+++ b/src/Splat/Splat.csproj
@@ -1,7 +1,6 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <TargetFrameworks>netstandard2.0;net5.0;net6.0</TargetFrameworks>
-    <TargetFrameworks Condition=" '$(OS)' == 'Windows_NT' ">$(TargetFrameworks);net461;net472</TargetFrameworks>
     <AssemblyName>Splat</AssemblyName>
     <RootNamespace>Splat</RootNamespace>
     <Authors>.NET Foundation and Contributors</Authors>
@@ -11,8 +10,5 @@
     <LangVersion>latest</LangVersion>
     <Nullable>enable</Nullable>
   </PropertyGroup>
-
-  <ItemGroup Condition=" $(TargetFramework.StartsWith('net46')) ">
-    <PackageReference Include="System.ValueTuple" Version="4.5.0" />
-  </ItemGroup>
+  
 </Project>