Figure out where rogue `Newtonsoft.Json` requirement is coming from about labs-windows HOT 6 OPEN

@michael-hawker Not sure just by having a quick look, I can say though that it's definitely not the Roslyn projects.

from labs-windows.

I don't know why it's needed, but removing it cause VS to hang when building the ALL solution

from labs-windows.

I wonder if it's some SourceGenerator/VS extension type dependency thing... But that'd be a bit odd. Any ideas @Sergio0694?

@Arlodotexe do you remember when you had to add it what was going on?

from labs-windows.

I'm actually testing having to add a dependency on this for the unit tests to work for UWP in #126... so maybe that's it. Why this is the case with the refactor vs. the original configuration, I'm not sure. Though we'll see if this works as I expect. Otherwise, I'm not sure what's going on.

from labs-windows.

This in part may be coming from our Unit Test projects, see microsoft/vstest#2488

from labs-windows.

Alright, I had changed the order of imports which made the reference to UnitTestPlatformVersion invalid, which was messing up the <SDKReference Include="TestPlatform.Universal, Version=$(UnitTestPlatformVersion)" /> line we had in our Labs.Tests.Uwp.props file.

Because of this, everything still worked, except for this implicit Newtonsoft.Json reference. By correcting the order of the imports and defining this property to make the SDK reference proper, it seems to correct the issue. Tested locally and in the CI.

Applying all these changes uniformly now for #126 for #124.

from labs-windows.