-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Support config binding ext methods in options assembly #87733
Conversation
Tagging subscribers to this area: @dotnet/area-extensions-configuration Issue DetailsThat is, intercept calls to methods on With this change, all the expected/common entry points for configuration binding in ASP.NET + AOT scenarios are now covered. cc @eerhardt @DamianEdwards. cc @andrewlock issues with #86348 should be fixed following this PR.
|
...icrosoft.Extensions.Configuration.Binder/gen/GeneratedMethods/ConfigBinderMethodSpec.Emit.cs
Outdated
Show resolved
Hide resolved
…her than new method spec types
e4a4b60
to
cc511b0
Compare
...braries/Microsoft.Extensions.Configuration.Binder/gen/Helpers/Emitter/ConfigurationBinder.cs
Show resolved
Hide resolved
...libraries/Microsoft.Extensions.Configuration.Binder/gen/Helpers/Emitter/CoreBindingHelper.cs
Show resolved
Hide resolved
...ibraries/Microsoft.Extensions.Configuration.Binder/gen/Helpers/Parser/ConfigurationBinder.cs
Show resolved
Hide resolved
...aries/Microsoft.Extensions.Configuration.Binder/gen/ConfigurationBindingGenerator.Emitter.cs
Outdated
Show resolved
Hide resolved
src/libraries/Microsoft.Extensions.Configuration.Binder/gen/Helpers/SourceWriter.cs
Outdated
Show resolved
Hide resolved
@andrewlock I verified that the scenarios in #86348 work following this PR. Please give it another go using our nightly runtime builds - |
Contributes to #83600.
Generator now intercepts calls to methods on
OptionsBuilderConfigurationExtensions
andOptionsConfigurationServiceCollectionExtensions
. This adds to existing support forConfigurationBinder
methods.With this change, all the expected/common entry points for configuration binding in ASP.NET + AOT scenarios are now covered. cc @eerhardt @DamianEdwards.
cc @andrewlock issues with #86348 should be fixed following this PR.