opserver / Opserver

Stack Exchange's Monitoring System
https://opserver.github.io/Opserver/
MIT License
4.51k stars 828 forks source link

Fix Microsoft.WebApplication.targets path #289

Closed rducom closed 6 years ago

rducom commented 6 years ago

Issue : Unable to use VS2k17 publish feature. This fix Microsoft.WebApplication.targets path in csproj

NickCraver commented 6 years ago

The current path is intentional, as it bypasses the requirement of needing full web application targets install on the build server for what amounts to a < 70KB .dll and .targets pair.

What are you hitting with the deploy in VS 2017?

rducom commented 6 years ago

I'm hitting the folowing error : The target "GatherAllFilesToPublish" does not exist in the project.

mqbk commented 6 years ago

how did you correct this error? I am getting the same thing.

NickCraver commented 6 years ago

Finally found time to fix our build agents (lacking the proper web workloads), merging in - and thanks for the reminder!