search

Half-Shot / matrix-dotnet-sdk

A matrix client library in .NET (C#)
MIT License
62 stars 23 forks source link

Is Microsoft.Extensions.Logging 3.0.0-preview.19074.2 necessary? #7

Open malachib opened 5 years ago

malachib commented 5 years ago

It doesn't seem to be, and is stopping me from compiling a netcore2.2 app

EDIT: I see that the LoggerFactory.Create method is indeed being used, so technically that makes a dependency. Can we find a different way so that we can rely on 2.2 versions of the logger?

Half-Shot commented 5 years ago

Yes sure! I will probably switch to a better logger (probs Serilog). I'm going to be a little busy for a while yet, so if you want to go ahead and remove the dependency (I don't think I make extensive use of the logger) and PR it then that would be cool :)

malachib commented 5 years ago

I instead encourage to further embrace Microsoft's logger interface (abstract interface only, Serilog underneath is a fine choice) - I'm sure we can make it gracefully live in your environment ! If I have time to do a PR definitely will. I tried removing the dependency, but need to fiddle with the factory more.

Also, separately, hit an issue with Client.Keys, wasn't present so wouldnt compile -- any guidance there?

Half-Shot commented 5 years ago

Yeah, it was missing some stuff, commited those.