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

Change promise usage to recommendation #49

Merged
merged 1 commit into from Oct 22, 2018
Merged

Change promise usage to recommendation #49

merged 1 commit into from Oct 22, 2018

Conversation

ghost
Copy link

@ghost ghost commented Oct 20, 2018

This PR is a split of the original PR #45 (the original PR shall stay open until completely done).

This PR changes the usage of promises inside this library to the recommendation listed on reactphp/promise. This PR also removes the unreachable code in AbstractSyncAdapter and adds exceptions to rejected promises.

The used promises in the tests are untouched, as they will be refactored anyway in a future PR.

@WyriHaximus WyriHaximus added this to the v0.1.2 milestone Oct 20, 2018
@jsor
Copy link
Member

jsor commented Oct 22, 2018

I'm not able to approve through the UI currently due to unicorns. So, oldschool :shipit: for now.

1 similar comment
@jsor
Copy link
Member

jsor commented Oct 22, 2018

I'm not able to approve through the UI currently due to unicorns. So, oldschool :shipit: for now.

Copy link
Member

@jsor jsor left a comment

Choose a reason for hiding this comment

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

👍

@WyriHaximus WyriHaximus merged commit 766cdef into reactphp:master Oct 22, 2018
@ghost ghost deleted the patch-promises branch October 22, 2018 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants