search

serilog / serilog-extensions-logging

Serilog provider for Microsoft.Extensions.Logging
Apache License 2.0
307 stars 97 forks source link

7.0.0 Release #223

Closed nblumhardt closed 1 year ago

nblumhardt commented 1 year ago
nblumhardt commented 1 year ago

This one's ready to go.

sungam3r commented 1 year ago

Wait for #219 ?

nblumhardt commented 1 year ago

Given the difficulty of getting everything into shape and out at the same time, I don't think we should hold up the release. There's always a 7.0.1, etc., to ship another day :-)

niemyjski commented 1 year ago

@nblumhardt any reason to pin to v7 instead of lts v6?

nblumhardt commented 1 year ago

@niemyjski just practicalities, really. Hopefully most people who really need LTS versions of everything will already be managing dependency upgrades carefully and can stick with whatever they're using today. Otherwise, v7 of these packages all include .NET 6 targets anyway.

There's also a new LTS just around the corner with v8, we've already published a Serilog.AspNetCore v6, and we don't want to have to coordinate shipping two new releases just to get the versioning policy bootstrapped. Did give it all lots of thought; always trade-offs whatever we do :-)

sungam3r commented 1 year ago

Do you intentionally not reference serilog v3 here and in other repos with such v7 releases?

nblumhardt commented 1 year ago

@sungam3r thanks for the double-check; yes, I think we still have some work to go on Serilog v3 👍

I'm not sure at which stage we'll want to flip over the dependencies here (and in the other projects), if Serilog v3 is out within a few months, the v8 packages might pull it in, perhaps.