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

[2.0] Started work on v2 #108

Open
wants to merge 1 commit into
base: master
from

Conversation

@GrahamCampbell
Copy link
Contributor

GrahamCampbell commented Jan 12, 2020

PHP 7.2+, with the same changes as I proposed for Guzzle 7, avoiding calling the helper functions within the core code, to mitigate global package installation issues.

@mtdowling

This comment has been minimized.

Copy link
Member

mtdowling commented Jan 12, 2020

Good ideas!

A random idea I had that I never had time to pursue was whether or not this library could move towards a thread-safe solution. As of right now, all promises mutate a global queue and can't be easily used with libraries like pthreads (anecdotally basing this on issues I've seen over the last few years).

.travis.yml Show resolved Hide resolved
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.