Conversation
7e1f4a4 to
a404424
Compare
|
Can you document how we know this doesn't reintroduce the problems we had with constructor injection before? #40521 |
|
@SteveSandersonMS I suspect is dotnet/runtime@a48e73a It's essentially using new APIs. I haven't done size testing myself, but given we are early in the release, I think we can make this go in, and observe if we find any regression in our regular runs. Are you ok with that? |
a404424 to
ca2bbf1
Compare
It's less for us to keep track of if you just do a one-off manual publish locally to see the impact on final sizes, otherwise if some regression appears two weeks from now, we have to go through a whole detective process to trace it back to the cause. |
I did measure the change for the Blazor hosted app in release configuration between main and this branch and the size is the same. |
* Adds support for constructor injection in components.
Fixes #18088
Reverts 87f870d