serilog / serilog-extensions-logging

Serilog provider for Microsoft.Extensions.Logging
Apache License 2.0
313 stars 100 forks source link

move to current sdk #138

Closed SimonCropp closed 5 years ago

nblumhardt commented 5 years ago

Looks like appveyor might not yet have that SDK installed? The build scripts, once upon a time, would install the target SDK, but I think life turned out simpler relying on an installed version :-)

SimonCropp commented 5 years ago

fixed it. TBH I find the use of global.json to be too restrictive for people contributing. i omit that file from my repos, and just ensure the CI server has a new enough SDK on it

nblumhardt commented 5 years ago

Thanks Simon! 👍