Explicit cs Implict reference of NETStandard.Library

highlight.xml

Are there any differences to the two approaches below? I.e impliclity ending up with the package reference to
NETStandard.Library 1.6.1, dvs running with DisableImplicitFrameworkReferences = true and adding an explicit
reference to NETStandard.Library?

Any pros/cons for either? Any hidden/implicit impacts?

See https://github.com/dotnet/sdk/pull/633#issuecomment-272515440 for some context

<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net452;netstandard1.6</TargetFramework>
  </PropertyGroup>

</Project>

vs.

<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net452;netstandard1.6</TargetFramework>
    <DisableImplicitFrameworkReferences>true</DisableImplicitFrameworkReferences>
  </PropertyGroup>

  <ItemGroup Condition=" '$(TargetFramework)' == 'netstandard1.6' ">
    <PackageReference Include="NETStandard.Library" Version="1.6.1" />
  </ItemGroup>

</Project>

I think you have the condition backwards, but if you want a cleaner tree in solution explorer, this would make sense:

<DisableImplicitFrameworkReferencesCondition=" '$(TargetFramework)' == 'net452' ">true</DisableImplicitFrameworkReferences>

@dsplaisted opps! Yeah that's what I meant.. a copy/paste mistake 😊

Just guessing with .NET Core 3, DisableImplicitFrameworkReferences == true and the fact it's not published on NuGet anymore doesn't go well together? Or is it still published?