serilog / serilog-extensions-logging

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

Does not work with the latest preview #116

Closed aeos closed 6 years ago

aeos commented 6 years ago

Hi,

I am trying to use the Microsoft.Extensions.Hosting package 2.1.0-preview1-final. and it uses the Microsoft.Extensions.Logging package of the same version. However when I add Serilog.Extensions.Logging 2.0.2 to my project I am unable to reference: use Serilog.Extensions.Logging;

Nor am I able to call AddSerilog() on the ILoggingBuilder

nblumhardt commented 6 years ago

Hi, thanks for the note! Any clues you can spot? There could be some info to be found in the output of dotnet restore, or the build process; nothing immediately comes to mind here (AFAIK there shouldn't be breaking changes that impact this library).

aeos commented 6 years ago

after much wrangling, I got it to work. All's well!

andymac4182 commented 6 years ago

What did you have to do to wrangle it @aeos ?

aeos commented 6 years ago

I am not convinced I need to do anything. Originally vscode was not triggering intellisense for addserilog and did not allow me to use serilog.extensions.logging. so I checked out this repository, changed the dependency and compiled a dll. I included it in my project and everything worked. I then removed the dll to confirm the fix, only to find that vscode had no problems with the original nuget package. So I am thinking it was a vscode gremlin.