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(ivy): remove load instruction #32067
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, thnx @crisbeto - it was on my TODO list!
If you into those type of changes we could also replace ɵɵloadViewQuery
and ɵɵloadContentQuery
with just one instruction ( ɵɵloadQuery
?) as the 2 mentioned functions are the same now.
Cool! Will do next week. |
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, one minor thing
9c54561
to
bb25717
Compare
I've reworked in based on the feedback @kara. |
Combines the `loadViewQuery` and `loadContentQuery` instructions since they have the exact same internal logic. Based on a discussion here: angular#32067 (review)
Combines the `loadViewQuery` and `loadContentQuery` instructions since they have the exact same internal logic. Based on a discussion here: #32067 (review) PR Close #32100
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
These changes remove the `ɵɵload` instruction which isn't being generated anymore.
bb25717
to
5ff529e
Compare
merge-assistance: global approval |
Combines the `loadViewQuery` and `loadContentQuery` instructions since they have the exact same internal logic. Based on a discussion here: angular#32067 (review) PR Close angular#32100
These changes remove the `ɵɵload` instruction which isn't being generated anymore. PR Close angular#32067
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. |
These changes remove the
ɵɵload
instruction which isn't being generated anymore.