Skip to content

Conversation

nblumhardt
Copy link
Member

@nblumhardt nblumhardt commented Nov 23, 2017

Plus build/documentation items #988, #994, #996, #1019, #842, #1042, #1050, #1055, #1063.

nblumhardt and others added 30 commits June 21, 2017 10:25
Travis CI Image - use edge group
Remove Rager releases badge
LogContext's remoting issue fix for .NET 4.5
- exit with non zero-code on error
Enable .NET Native on AppVeyor + workaround on compiler error for #993
Code of Conduct, Contributing Guide & templates for issues and PRs [Skip CI]
The link in PULL_REQUEST_TEMPLATE.md pointed to a 404 (https://github.com/serilog/serilog/CONTRIBUTING.md)
Meta PR to fix PR Template [Skip CI]
tsimbalar and others added 24 commits October 16, 2017 07:54
Protect against possible null from base class
When a parameter is an abstract class, now accept the type name of a concrete subclass that has a default constructor ( the same way interfaces are supported)
Refactored KeyValuePairSettings to accept a IReadOnlyDictionary<string, string> instead of IEnumerable<KeyValuePair<string, string>> in order to reveal the fact that key-value settings are not lazily consumed. 

Added tests on edge cases : duplicate keys, irrelevant settings.
Instead, take the last value of each key, to remove work from future Serilog.Settings.Combined
Add support for abstract class parameters in settings
Handle duplicate key-value pair settings - last-in wins
Move LoggerExtension so that namespace and folder matches
as a value for parameters, using the syntax `NameSpace.To.ConcreteType::StaticProperty, AssemblyName`
Add support for reference to public static properties in settings
@tsimbalar
Copy link
Member

tsimbalar commented Nov 24, 2017

🎉
A few comments :

@nblumhardt
Copy link
Member Author

Fixed, thanks @tsimbalar 👍

@nblumhardt nblumhardt changed the title [WIP] 2.6.0 Release 2.6.0 Release Dec 4, 2017
@nblumhardt nblumhardt merged commit 9a3b037 into master Dec 4, 2017
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.

7 participants