Skip to content

Conversation

nblumhardt
Copy link
Member

@nblumhardt nblumhardt commented Jan 28, 2018

nblumhardt and others added 27 commits December 4, 2017 14:13
Closes #1073 - Overload Wrap() to accept LogEventLevel and LoggingLevelSwitch.
…t templates; fixes #1088 - handle positional properties when computing {Properties}
Improvements to {Properties} handling in output templates
Fixes #1105 - add a static None property to the Logger class
- force schedule async continuations on a new thread
`ContextPropertiesCrossAsyncCalls` test failure fix on CI:
Update of README badges to include downloads and tests. [Skip CI]
@tsimbalar
Copy link
Member

I'm not sure why, but the current pre-release (dev) nuget packages are versioned v2.7.1 ... so I guess this release should be v2.7.1, to avoid going through decreasing versions ?

@adamchester
Copy link
Member

adamchester commented Mar 20, 2018

Why isn't the Travis build status part of the github checks here?

image

merbla and others added 21 commits March 21, 2018 09:22
SourceLink v2 support added
…fied in #1154

 - Ensures that `wt.X(); wt.Y()` and `wt.X().WriteTo.Y()` are equivalent
 - Ensures a failing sink doesn't block events from reaching others inside the same wrapper, much as using a secondary logger would do
 - Prevents chained calls from accidentally breaking thread-safety rules by turning `Enrich` inside wrappers into a no-op
Created Logger should not hold reference to its configuration
Fixes a number of issues with `LoggerSinkConfiguration.Wrap()`
@nblumhardt nblumhardt changed the title [WIP] Release 2.7.0 Release 2.7.0 May 17, 2018
@nblumhardt
Copy link
Member Author

I've updated the release notes above and think we've got enough here to warrant merging/moving forward.

Has anyone had an opportunity to put 2.7-dev through its paces in a more demanding setting? I should have some time in the next few days to update various apps, but it'd be great if more people can smoke test, too.

@merbla merbla changed the title Release 2.7.0 Release 2.7.1 May 17, 2018
@merbla
Copy link
Contributor

merbla commented May 17, 2018

LGTM, as noted it will be 2.7.1 and with existing packages on NuGet not sure it is worth bumping it back.

@nblumhardt nblumhardt merged commit 12ca059 into master May 18, 2018
@nblumhardt
Copy link
Member Author

This is now published! 🎉

One small caveat encountered while updating Serilog.Sinks.Async post-publishing, the implicit reference to NETStandard.Library that's now used for the .NET Standard targets implicitly bumps the versions of framework dependencies up. The change will fix DLL hell for some, and require others to upgrade or explicitly specify dependency versions. The world we live in...

Binary compatibility is not impacted, and neither are .NET Framework builds, so fingers crossed this'll all come out fine 😅 .

@tsimbalar
Copy link
Member

Updated in https://github.com/tsimbalar/serilog-settings-comparison , all tests are green (not that I expected otherwise :P ).

@tsimbalar
Copy link
Member

Also, it turns out this release was build number 999 for AppVeyor ... I hope the next build will not break everything à la Y2K bug :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants