Thanks, LGTM.

@mellinoe Looks like the reason why CI broke with this new version of buildtools is that when this code was added dotnet/buildtools#655 it brought to light some issues that coreclr repo had but where very luckily working before. For example, non of the tests assemblies where being signed even though the IsTestProject variable wasn't defined, and this was because this line was being overriden later in the import tree by the VS MSBuild targets. When you added IL.targets, this line imports the VS targets first, so now the test projects are failing when they try to be signed.

I guess a possible fix would be to disable the signing by setting <SkipSigning>true</SkipSigning> but even if that works the new imported IL.targets will conflict with the coreclr's IL.targets so we that might add extra errors/warnings.