Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Add support for builder.AddAppSettings #9

Merged
merged 3 commits into from
Mar 4, 2018

Conversation

tsimbalar
Copy link
Contributor

First part of #5 : AppSettings

Support for loading key value pairs from an arbitrary config file.

Notes :

  • The AppSettingsSettings.cs file has been copied and adapted from Serilog.Settings.AppSettings for now.
  • All the AppSettings related part is surrounded with #if APPSETTINGS to make it available only on full framework for now. I don't know if that could be problematic and if there is a better way
  • This is a "repackaging" of a bigger bunch of commits, and to make things easier, I have put all the creation of Test support stuff (TestDummies, Tests.Support etc) in the first commit.

@nblumhardt nblumhardt merged commit 03a2181 into serilog-archive:dev Mar 4, 2018
@nblumhardt
Copy link
Contributor

👍 thanks for the split PRs, a bit easier to digest :-)

@tsimbalar tsimbalar deleted the addappsettings branch March 5, 2018 07:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants