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
refactor(core): deprecate entryComponents #33205
refactor(core): deprecate entryComponents #33205
Conversation
8c02a45
to
005d93d
Compare
You can preview bf7cc34 at https://pr33205-bf7cc34.ngbuilds.io/. |
I think we should also deprecate Also, could you fix the commit message
|
005d93d
to
fd545ac
Compare
The feedback has been addressed. |
You can preview fd545ac at https://pr33205-fd545ac.ngbuilds.io/. |
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, with minor nits
With Ivy the `entryComponents` array isn't necessary anymore. These changes mark it as deprecated so that it can be removed in a future version.
fd545ac
to
c63a8b6
Compare
The PR has been addressed. |
You can preview c63a8b6 at https://pr33205-c63a8b6.ngbuilds.io/. |
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! thanks
With Ivy the `entryComponents` array isn't necessary anymore. These changes mark it as deprecated so that it can be removed in a future version. PR Close angular#33205
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. |
With Ivy the
entryComponents
array isn't necessary anymore. These changes mark it as deprecated so that it can be removed in a future version.