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
1.1 release for 2.0 compatibility #31
Conversation
namespace React\Promise; | ||
|
||
if (function_exists('React\Promise\resolve')) { | ||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC PHP only supports conditional definitions if they are actually wrapped in conditionals? In other words, has this very statement been tested? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, fixed 😁
Awesome, I'd love to get this in! 👍 This certainly helps migrating from legacy PHP versions and the v1 Promise API.
Personally, I'd vote to postpone this for now, so we can get this out quicker. Should we actually need these in the future, we can still push them with an eventual v1.2 release. Other than my above remark, this LGTM 👍 |
👍 on postponing the optional todos |
+1 |
👍 Let's get this in! :) |
This is hugely helpful, thanks! |
This could become a 1.1 release which makes the API more compatible with 2.0 while preserving full backward compatibility. With this release, consuming packages can require both 1.0 (with PHP 5.3 support) and 2.0.
Changes:
Optional todos: