Skip to content
New issue

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

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

Already on GitHub? # to your account

Update Instructions for .NET Core 3 templates #167

Merged
merged 1 commit into from
Jan 24, 2020

Conversation

phlashdev
Copy link
Contributor

Fixes #147

The instructions for using Serilog contained statements that were no longer valid since .NET Core 3.0 uses the Generic Host and new project templates.

The code sample in the readme and the corresponding statements were updated to resemble these changes.

The instructions for using Serilog contained statements that were no longer valid since .NET Core 3.0 uses the Generic Host and new project templates.

The code sample in the readme and the corresponding statements were updated to resemble these changes.
@phlashdev
Copy link
Contributor Author

I think that probably it would be a good idea to inlcude the instructions for users that are still on pre .NET Core 3 versions. But as it seems to me the readme file resembles the instructions for the latest version, and as the templates vary also for example from .NET Core 2.0 to 2.1, this could make the readme somewhat unclear?

I'm a little unsure about we should do and how we do it, so I really would like to know what you're thinking.

@phlashdev phlashdev requested a review from nblumhardt January 23, 2020 14:09
@nblumhardt
Copy link
Member

I think it's worthwhile moving the examples forwards, since 3.x is now well-and-truly established. Thanks!

@nblumhardt nblumhardt merged commit 21c14f6 into serilog:dev Jan 24, 2020
@nblumhardt nblumhardt mentioned this pull request Jul 24, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with this: Remove ILoggerFactory parameters and any Add*() calls on the logger factory in Startup.cs
2 participants