Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

fix(watch_group): fix for NaN !== NaN #1146

Closed
wants to merge 2 commits into from
Closed

fix(watch_group): fix for NaN !== NaN #1146

wants to merge 2 commits into from

Conversation

vicb
Copy link
Contributor

@vicb vicb commented Jun 11, 2014

closes #1139

@vicb vicb added cla: yes and removed cla: no labels Jun 11, 2014
@vicb vicb added this to the 0.14.0 milestone Jun 25, 2014
@mhevery
Copy link
Contributor

mhevery commented Jul 21, 2014

@vicb can we use the looseIdentical from the other PR#1149 there?

@vicb
Copy link
Contributor Author

vicb commented Jul 22, 2014

You're right. Let's merge #1149 first and then I'll update this PR to use looseIdentical.

@vicb vicb closed this in d24ff89 Jul 23, 2014
vicb added a commit that referenced this pull request Jul 23, 2014
@vicb @chirayuk
fix(watch_group): fix for NaN !== NaN
e02bab6 
closes #1139

Closes #1146
@vicb vicb deleted the 0611-wgNaN branch July 31, 2014 08:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
api: no-change cla: yes type: bug
Milestone
0.14.0
Development

Successfully merging this pull request may close these issues.

[WatchGroup] Handling of NAN
2 participants
@vicb @mhevery