-
Notifications
You must be signed in to change notification settings - Fork 73
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
Please change the namespace of the Configuration extensions methods #140
Comments
This is a breaking change. We can still take it at this point but we will need to weigh out the pros and cons on this. |
@shibayan Thanks for the feedback. Since this is a breaking change, it might take a while, but we will include it in our next major release for the configuration provider. |
@abhilasharora I think that Breaking change is allowed while previewing. You actually have deleted the Watch / WatchAndReloadAll method. It was quite a breaking change for me |
@shibayan I believe we moved from 1.x.x to 2.x.x with that change though? We certainly can make the change when we go to 3.x.x |
@jimmyca15 Umm. In general, don't change the major version in preview. It is strange that the SDK version is v2.x.x or v3.x.x in the first release of the service 😕 I think it is important for usability to follow general rules. |
@shibayan We just released version |
As a convention of .NET Core / ASP.NET Core, the extension method namespace is generalized.
Azure App Configuration (2.0.0-preview-009470001-1371)
Example: Key Vault
I'd like you to change the namespace to match other Configuration libraries.
The text was updated successfully, but these errors were encountered: