richardszalay / helix-publishing-pipeline

Unified publishing for Sitecore Helix solutions that extends existing Visual Studio and command line workflows
MIT License
68 stars 18 forks source link

PackageReference compatiblity #56

Closed richardszalay closed 5 years ago

richardszalay commented 5 years ago

54 made me realise that there could be a number of issues that arise from running via PackageReference since the targets file is include before the Web.Application.targets so patching XXXDependsOn might result in things running too early.

I was right and two issues arose:

This PR resolves both these issues and moves the test fixture import before the Web.Application.targets import in order to raise any future compatibility issues that might crop up.