Skip to content

feat(CD): add support for === and !== - #1983

Merged
vicb merged 1 commit into
angular:masterfrom
vicb:0519-cdIdentical
May 20, 2015
Merged

vicb merged 1 commit into
angular:masterfrom
vicb:0519-cdIdentical

Conversation

@vicb

@vicb vicb commented May 19, 2015

Copy link
Copy Markdown
Contributor

relates to #1500

@pkozlowski-opensource you PR add support in the parser but not in the CD (which I think was the root cause of issue #1496).

Could you please review the PR ?

@vicb vicb added feature Label used to distinguish feature request from other issues #change_detection labels May 19, 2015
@vicb

vicb commented May 19, 2015

Copy link
Copy Markdown
Contributor Author

Travis failure is not related to the change (#1982)

edit: rebased on top of the PR

@vicb
vicb force-pushed the 0519-cdIdentical branch 3 times, most recently from 8e78d58 to 1f2986a Compare May 19, 2015 11:58
@naomiblack naomiblack added this to the M10: Fixit milestone May 19, 2015
@pkozlowski-opensource

Copy link
Copy Markdown
Member

@vicb this LGTM based of my (very) limited knowledge of this code part. Thnx for taking care of this, I wasn't aware that we need to change those parts here as well. 馃憤

@vicb
vicb force-pushed the 0519-cdIdentical branch from 1f2986a to 0ae89ac Compare May 20, 2015 05:50
@vicb
vicb merged commit 0ae89ac into angular:master May 20, 2015
@vicb vicb removed the in progress label May 20, 2015
@vicb
vicb deleted the 0519-cdIdentical branch May 20, 2015 05:51
@angular-automatic-lock-bot

Copy link
Copy Markdown

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 Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes feature Label used to distinguish feature request from other issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants