Skip to content

Fix coroutine executor when using with promise#486

Merged
vladar merged 2 commits intowebonyx:0.13.xfrom
mcg-web:fix-coroutine-when-using-promise
Jun 6, 2019
Merged

Fix coroutine executor when using with promise#486
vladar merged 2 commits intowebonyx:0.13.xfrom
mcg-web:fix-coroutine-when-using-promise

Conversation

@mcg-web
Copy link
Contributor

@mcg-web mcg-web commented Jun 2, 2019

When trying to support experimental coroutine executor (#397) in OverblogGraphBundle the integration fails when using promises.

@mcg-web mcg-web changed the title Add a test to cover promises Fix coroutine when using with promise Jun 2, 2019
@mcg-web mcg-web changed the title Fix coroutine when using with promise Fix coroutine executor when using with promise Jun 2, 2019
@mcg-web mcg-web force-pushed the fix-coroutine-when-using-promise branch from 5e8bc91 to 08d9493 Compare June 2, 2019 18:49
@vladar
Copy link
Member

vladar commented Jun 5, 2019

@jakubkulhan Can you please review?

Copy link
Contributor

@jakubkulhan jakubkulhan left a comment

Choose a reason for hiding this comment

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

LGTM

@vladar vladar merged commit bd02ccd into webonyx:0.13.x Jun 6, 2019
@mcg-web mcg-web deleted the fix-coroutine-when-using-promise branch June 6, 2019 10:17
@mcg-web
Copy link
Contributor Author

mcg-web commented Jun 7, 2019

Thanks for review @jakubkulhan and @vladar for merging this. Now my tests are OK with 0.13.x dev branch can you please release a patch version @vladar ? What about the fix on master ?

@vladar
Copy link
Member

vladar commented Jun 12, 2019

@mcg-web Just published v0.13.5. I'll merge it to master branch.

And curious to see your results on experimental executor

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.

3 participants