-
Notifications
You must be signed in to change notification settings - Fork 26.3k
fix(core): remove forceRoot
flag for effects
#60535
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It will need public API goldens update, though (see failing CI)
11f9dcb
to
4e50ac1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Reviewed-for: public-api
Reviewed-for: fw-general
This flag is effectively unused in Angular code that we've seen, and is only serving to complicate the mental model of effects. It could be reintroduced if needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-for: public-api
4e50ac1
to
34351b4
Compare
This PR was merged into the repository by commit 43cbc58. The changes were merged into the following branches: main |
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. |
This flag is effectively unused in Angular code that we've seen, and is only serving to complicate the mental model of effects. It could be reintroduced if needed.