-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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(core): fix interpolate identifier in AOT #30243
Conversation
This commit fixes a regression introduced in PR 29692 where the interpolate symbol in View Engine was improperly prefixed with the ɵɵ that signifies private instructions for Ivy. It resulted in interpolations of 10+ values not working correctly in AOT mode. This commit removes the prefix.
This commit fixes a regression introduced in PR 29692 where the interpolate symbol in View Engine was improperly prefixed with the ɵɵ that signifies private instructions for Ivy. It resulted in interpolations of 10+ values not working correctly in AOT mode. This commit removes the prefix. PR Close #30243
@@ -212,4 +212,22 @@ describe('property instructions', () => { | |||
|
|||
expect(img.src.indexOf('unsafe:')).toBe(0); | |||
}); | |||
|
|||
it('should handle interpolations with 10+ values', () => { |
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.
@kara test description says 10+ values
, but only checks for exactly 10 values.
This commit fixes a regression introduced in PR 29692 where the interpolate symbol in View Engine was improperly prefixed with the ɵɵ that signifies private instructions for Ivy. It resulted in interpolations of 10+ values not working correctly in AOT mode. This commit removes the prefix. PR Close angular#30243
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 commit fixes a regression introduced in PR 29692 where
the interpolate symbol in View Engine was improperly prefixed
with the ɵɵ that signifies private instructions for Ivy. It
resulted in interpolations of 10+ values not working correctly
in AOT mode. This commit removes the prefix.
(I'm unable to add a good test for AOT given the way VE tests
are set up, but I've added one for JIT).