Skip to content
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(compiler): do not error if $any is used inside a listener #43866

Closed
wants to merge 1 commit into from

Conversation

crisbeto
Copy link
Member

Fixes that $any casts weren't being converted to statements inside listeners which resulted in a compiler error.

Fixes #43841.

@google-cla google-cla bot added the cla: yes label Oct 17, 2021
@crisbeto crisbeto marked this pull request as ready for review October 17, 2021 10:49
@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer area: compiler Issues related to `ngc`, Angular's template compiler target: patch This PR is targeted for the next patch release labels Oct 17, 2021
@ngbot ngbot bot modified the milestone: Backlog Oct 17, 2021
@JoostK JoostK removed the request for review from AndrewKushnir October 17, 2021 15:37
@JoostK JoostK added action: merge The PR is ready for merge by the caretaker action: presubmit The PR is in need of a google3 presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 17, 2021
@petebacondarwin
Copy link
Member

Shouldn't the commit messages be:

fix(compiler): support $any is used inside a listener 

or

fix(compiler): do not error if $any is used inside a listener 

??

As it is written, it sounds like the aim is to throw an error if $any is used in a listener.

@petebacondarwin petebacondarwin added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Oct 18, 2021
Fixes that `$any` casts weren't being converted to statements inside listeners which resulted in a compiler error.

Fixes angular#43841.
@crisbeto crisbeto changed the title fix(compiler): error if $any is used inside a listener fix(compiler): do not error if $any is used inside a listener Oct 18, 2021
@crisbeto crisbeto removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Oct 18, 2021
@AndrewKushnir
Copy link
Contributor

Presubmit.

@AndrewKushnir AndrewKushnir added target: rc This PR is targeted for the next release-candidate and removed action: presubmit The PR is in need of a google3 presubmit target: patch This PR is targeted for the next patch release labels Oct 18, 2021
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit c944fee.

AndrewKushnir pushed a commit that referenced this pull request Oct 18, 2021
Fixes that `$any` casts weren't being converted to statements inside listeners which resulted in a compiler error.

Fixes #43841.

PR Close #43866
@AndrewKushnir
Copy link
Contributor

@crisbeto there was a merge conflict with the 12.2.x branch, so this change was merged to 13.0.x and master branches. If the fix needs to be included into 12.2.x branch, could you please create a new PR with the necessary changes? Thank you.

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Nov 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: compiler Issues related to `ngc`, Angular's template compiler cla: yes target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: stmt.visitStatement is not a function
4 participants