`IncludeContentInPack` shouldn't affect `None` items

[Youssef1313]

Having <None Include="path/to/icon.png" Pack="true" PackagePath="icon.png" /> along with IncludeContentInPack set to false doesn't include the icon.png file in the package.

I don't think IncludeContentInPack should affect None items, especially when Pack is explicitly set to true.

[nkolev92]

As suggested by @Youssef1313, the behavior is that if the prop is set, none of the items are being considered:

https://github.com/NuGet/NuGet.Client/blob/55728432057dc57733661fe01e9c8493ab5034f2/src/NuGet.Core/NuGet.Build.Tasks.Pack/NuGet.Build.Tasks.Pack.targets#L491

Marking this as a DCR for pack.

[jeffkl]

Team Triage: Assigning to Heng since this can affect READMEs, icons, etc.