-
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
Fixes to and resolution of root cause failures in core animation tests #28026
Conversation
7c1076f
to
5561b5b
Compare
You can preview 7c1076f at https://pr28026-7c1076f.ngbuilds.io/. |
You can preview 5561b5b at https://pr28026-5561b5b.ngbuilds.io/. |
5561b5b
to
24d5024
Compare
You can preview 24d5024 at https://pr28026-24d5024.ngbuilds.io/. |
// setProperty without a value doesn't make any sense | ||
if (value.name || value.value) { | ||
const bindingName = prepareSyntheticPropertyName(input.name); | ||
const hasValue = value && (value instanceof LiteralPrimitive) ? !!value.value : true; |
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.
Can you add a comment here to explain this check?
24d5024
to
a7de6cf
Compare
You can preview a7de6cf at https://pr28026-a7de6cf.ngbuilds.io/. |
…empty bindings
You can preview 184f500 at https://pr28026-184f500.ngbuilds.io/. |
…empty bindings
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
You can preview 815b842 at https://pr28026-815b842.ngbuilds.io/. |
@matsko new g3 presubmit (the old one has conflicts, since the g3 sync CL just landed): |
Thank you @AndrewKushnir |
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. |
No description provided.