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 #404 Updated IdenticalOperator.php and operators.h #423

Merged
merged 2 commits into from Jun 18, 2014

Conversation

@dreamsxin
Copy link
Member

@dreamsxin dreamsxin commented Jun 14, 2014

No description provided.

@nkt
Copy link
Contributor

@nkt nkt commented Jun 14, 2014

@dreamsxin please add tests for this.

@nkt nkt added the not tested label Jun 14, 2014
@dreamsxin
Copy link
Member Author

@dreamsxin dreamsxin commented Jun 14, 2014

@nkt Thanks

@nkt
Copy link
Contributor

@nkt nkt commented Jun 14, 2014

@dreamsxin thank you!👍

@nkt nkt removed the not tested label Jun 14, 2014
phalcon pushed a commit that referenced this pull request Jun 18, 2014
Fix #404 Updated IdenticalOperator.php and operators.h
@phalcon phalcon merged commit d1f35e1 into phalcon:master Jun 18, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.