Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build script for OSX/Nix & Samples update #97

Merged
merged 5 commits into from
Aug 23, 2017

Conversation

merbla
Copy link
Contributor

@merbla merbla commented Aug 22, 2017

Follow on from #95. Current example does not illustrate 2.0 release changes.

  • Removed previous sample
  • Added two new samples for vanilla web and WebAPI.

@merbla
Copy link
Contributor Author

merbla commented Aug 22, 2017

Not sure but we may want to include https://github.com/serilog/serilog-extensions-logging/blob/dev/samples/WebSample/appsettings.json in the samples.

@merbla
Copy link
Contributor Author

merbla commented Aug 23, 2017

These samples may need to be refined see https://github.com/aspnet/Logging/issues/670#issuecomment-324174361

@nblumhardt
Copy link
Member

Looks good; updates can probably wait until we've dug deeper into the 2.0 issues. Ready to :shipit: ?

@merbla
Copy link
Contributor Author

merbla commented Aug 23, 2017

:shipit: This should provide a good playground for the issues in #98

@nblumhardt nblumhardt merged commit 88991c6 into serilog:dev Aug 23, 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.

None yet

2 participants