You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There may be bad UX edge cases that need to be investigated, such as :
The roslyn analyzers report an unused using-import from where the existing extension methods are (might be topically unlikely)
write the smallest possible source file that could invoke each hook and make sure that we don't have unused usings after source generator hooks the callsite.
Concerns around callers using static invocation syntax instead of instance/extension syntax.
feature not a bug
make sure it triggers linker warning
have documentation on that linker warning that leads people to using the generator.
The generator might end up toggling itself on and off (by removing the calls to the existing methods)
have a discussion and review the triggering logic with folks from the compiler team
The text was updated successfully, but these errors were encountered:
Tagging subscribers to this area: @dotnet/area-extensions-configuration
See info in area-owners.md if you want to be subscribed.
Issue Details
[WIP] Placeholder for follow up work for the upcoming ConfigurationBinder source generator (#44493).
The task is to make all the ConfigurationBinder tests (which are now shared with the reflection-based implementation) pass. Identified work items include:
Implement the right constructor invocation logic for each POCO-style target config type
layomia
changed the title
Make ConfigurationBinder source generator functionally feature-complete
Make ConfigurationBinder source generator feature-complete
Feb 16, 2023
Follow up work for the configuration binding source generator (#44493, #82179).
Preview 3
Preview 4
EnableConfigurationBinderGenerator
) to serve as gesture to invoke generator (e.g. EnableRequestDelegateGenerator flag) (as part of How should we represent source-generators / analyzers that are off by default? sdk#30872) and include generator in the ASP.NET Targeting PackPreview 5
Preview 6
Preview 7
RC 1
Stretch / tbd / validation
FX DC feedback
There may be bad UX edge cases that need to be investigated, such as :
The text was updated successfully, but these errors were encountered: