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

Move extensions to Microsoft.Extensions.DependencyInjection namespace #421

Closed
304NotModified opened this issue May 6, 2020 · 3 comments
Closed

Comments

@304NotModified
Copy link
Member

304NotModified commented May 6, 2020

As this is the recommendation nowadays

image

https://docs.microsoft.com/en-us/aspnet/core/fundamentals/dependency-injection?view=aspnetcore-3.1

@304NotModified
Copy link
Member Author

e.g. LoadConfigurationFromSection , AddNLogWeb

@304NotModified
Copy link
Member Author

I really should create a PR for this :)

@304NotModified
Copy link
Member Author

We have already 3 namespaces

using NLog;
using NLog.Extensions.Logging;
using NLog.Extensions.Hosting;

and we could not move them all to Microsoft.Extensions.DependencyInjection, so this isn't really the way

@304NotModified 304NotModified removed this from the 5.0 milestone Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant