-
Notifications
You must be signed in to change notification settings - Fork 24.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
fix(forms): _reduceValue
arrow function now has correct types.
#44483
Conversation
I previously strengthened some weak types in angular#44370. One of these fixes exposed an incorrect call into `_reduceChildren` from `_reduceValue`. This was caught in google3 by a caller who was extending `FormGroup` and overriding these methods. Special thanks to Bart G for catching this issue and suggesting a fix.
4fff3f6
to
879dca0
Compare
Rerunning failures reveals these are flakes, as most of the targets go green: https://test.corp.google.com/OCL:416417012:BASE:416422956:1639528317533:31818db4 |
This PR was merged into the repository by commit 6a0dadf. |
) I previously strengthened some weak types in #44370. One of these fixes exposed an incorrect call into `_reduceChildren` from `_reduceValue`. This was caught in google3 by a caller who was extending `FormGroup` and overriding these methods. Special thanks to Bart G for catching this issue and suggesting a fix. PR Close #44483
…ular#44483) I previously strengthened some weak types in angular#44370. One of these fixes exposed an incorrect call into `_reduceChildren` from `_reduceValue`. This was caught in google3 by a caller who was extending `FormGroup` and overriding these methods. Special thanks to Bart G for catching this issue and suggesting a fix. PR Close angular#44483
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
I previously strengthened some weak types in #44370. One of these fixes exposed an incorrect call into
_reduceChildren
from_reduceValue
. This was caught in google3 by a caller who was extendingFormGroup
and overriding these methods.Special thanks to Bart G for catching this issue and suggesting a fix.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The type of the callback in
_reduceValue
is incorrect, causing warnings for some users in google3.Issue Number: N/A
What is the new behavior?
The type of the callback is fixed.
Does this PR introduce a breaking change?
Other information