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

Cache #105

Merged
merged 8 commits into from
Dec 26, 2012
Merged

Cache #105

merged 8 commits into from
Dec 26, 2012

Conversation

igorw
Copy link
Contributor

@igorw igorw commented Dec 1, 2012

No description provided.

@igorw igorw mentioned this pull request Dec 1, 2012
$cachedRecords = $cache->lookup($query);
$promise = $cache->lookup($query);

$this->assertInstanceOf('React\Promise\FulfilledPromise', $promise);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should not check against React\Promise\FulfilledPromise as it's considered internal api

* master:
  Add libevent timers fix to CHANGELOG
  Whitespace nitpick
  force travis retest.  Appears 5.3.3 timed out
  5.3 backward compatibility for ->timersGc
  moved timersgc foreach loop into callback
  Fix timersGc iteration in LibEventLoop
  issue 76 fix.  Using SplQueue for
  Remove $loop arg from README example (thanks @DaveRandom)
  Add Response end event to CHANGELOG
  Tweak previous merge
  React\Http\Response emits 'end' event when stream is ended
@igorw
Copy link
Contributor Author

igorw commented Dec 24, 2012

This is now ready for merge.

@cboden cboden merged commit dce644c into master Dec 26, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants