Skip to content

Conversation

JeanMeche
Copy link
Member

@JeanMeche JeanMeche commented Oct 30, 2024

This commit adds the support for the in keyword as a relational operator, with the same precedence as the other relational operators (<,>, <=, >=).

As a bigger picture, adding this brings us toward

@pullapprove pullapprove bot requested a review from AndrewKushnir October 30, 2024 17:31
@angular-robot angular-robot bot added the area: compiler Issues related to `ngc`, Angular's template compiler label Oct 30, 2024
@ngbot ngbot bot added this to the Backlog milestone Oct 30, 2024
@JeanMeche JeanMeche requested review from alxhub and crisbeto and removed request for AndrewKushnir October 30, 2024 17:33
@angular-robot angular-robot bot added area: compiler Issues related to `ngc`, Angular's template compiler and removed area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 30, 2024
@angular-robot angular-robot bot added area: compiler Issues related to `ngc`, Angular's template compiler and removed area: compiler Issues related to `ngc`, Angular's template compiler labels Oct 31, 2024
@JeanMeche JeanMeche modified the milestones: Backlog, v20 candidates Oct 31, 2024
@JeanMeche JeanMeche force-pushed the in-operator branch 3 times, most recently from 4e8c96a to 8041fb9 Compare February 23, 2025 17:52
@JeanMeche JeanMeche changed the title refactor(compiler): support the in keyword in Binary expression feat(compiler): support the in keyword in Binary expression Feb 23, 2025
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Feb 23, 2025
@JeanMeche JeanMeche added feature Issue that requests a new feature breaking changes labels Feb 23, 2025
@angular-robot angular-robot bot added the detected: breaking change PR contains a commit with a breaking change label Feb 23, 2025
@JeanMeche JeanMeche removed detected: feature PR contains a feature commit detected: breaking change PR contains a commit with a breaking change labels Feb 23, 2025
@angular-robot angular-robot bot added detected: breaking change PR contains a commit with a breaking change detected: feature PR contains a feature commit labels Feb 23, 2025
@JeanMeche JeanMeche force-pushed the in-operator branch 2 times, most recently from 7ff4cb0 to b6c32c3 Compare March 28, 2025 13:20
@JeanMeche JeanMeche force-pushed the in-operator branch 4 times, most recently from 8d47aae to 111aaff Compare April 14, 2025 17:27
This commit adds the support for the `in` keyword as a relational operator, with the same precedence as the other relational operators (<,>, <=, >=)

BREAKING CHANGE: 'in' in an expression now refers to the operator
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release labels Apr 22, 2025
@JeanMeche JeanMeche removed the request for review from alxhub April 22, 2025 14:01
@pkozlowski-opensource
Copy link
Member

This PR was merged into the repository by commit 1b8e7ab.

The changes were merged into the following branches: main

@JeanMeche JeanMeche deleted the in-operator branch April 22, 2025 20:25
@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 May 23, 2025
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 breaking changes detected: breaking change PR contains a commit with a breaking change detected: feature PR contains a feature commit feature Issue that requests a new feature target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants