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(common): use transform
functions in NgOptimizedImage inputs
#50580
Conversation
241ca9d
to
c9efb02
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
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-common
LGTM but the CI is not happy. I was restarting the job assuming it is a flake but it seems like there is a real issue here, hence this PR will need cleanup. |
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
c9efb02
to
51ae5ee
Compare
@crisbeto it looks like the error on CI is legit and is related to the JIT mode and input transformations. I've got the error here:
and at the callback fn used as the first argument, I get an |
@AndrewKushnir I'll take a look tomorrow. Interesting that this didn't come up in the router PR though #50589 |
…in JIT mode Fixes an error that surfaced in angular#50580 where the compiler was throwing an error in JIT mode when reading the result of `compileDirectiveDeclaration`. It is caused by the fact that input transform functions were being passed around directly, instead of being wrapped in an AST node.
The error will be fixed by #50600. |
…in JIT mode (#50600) Fixes an error that surfaced in #50580 where the compiler was throwing an error in JIT mode when reading the result of `compileDirectiveDeclaration`. It is caused by the fact that input transform functions were being passed around directly, instead of being wrapped in an AST node. PR Close #50600
This commit refactors the code of NgOptimizedImage directive to switch from getter/setter approach to convers inputs to use the `transform` function instead.
51ae5ee
to
dffc8c7
Compare
Caretaker note: presubmit is "green", this PR is ready for merge. |
This PR was merged into the repository by commit cefa3de. |
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. |
angular#50580) This commit refactors the code of NgOptimizedImage directive to switch from getter/setter approach to convers inputs to use the `transform` function instead. PR Close angular#50580
angular#50580) This commit refactors the code of NgOptimizedImage directive to switch from getter/setter approach to convers inputs to use the `transform` function instead. PR Close angular#50580
This commit refactors the code of NgOptimizedImage directive to switch from getter/setter approach to convert inputs to use the
transform
function instead.PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?