-
Notifications
You must be signed in to change notification settings - Fork 793
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
Switch to become a ES7 async/await-style generator function wrapper #154
Conversation
LGTM! |
@tj can you add me on npm? |
@jonathanong added |
Change a lot |
Should this be released after node 0.12? How's the |
what do you mean? v0.11 already has promises, though it might not be ideal. I think the latest version has an okay implementation. hopefully they'll include the version of node that has generators enabled in v0.12 |
ignore me. nodejs/node-v0.x-archive#7714 already landed in 0.11.14. |
does this maintain backwards compatibility or no? |
nvm, looks like it doesnt. read some of the readme |
better version of #134. now it's future-proof with async/await, and i also got 100% coverage!
only issue is that the browser is now required to have a Promise implementation, so we run into the same #149 problem, but i don't think we should bother solving that because if a platform supports generators, it'll support promises as well.
i'll update the examples, readme, and wiki if we decide to merge this. this would be v4
issue bookkeeping: closes #149 closes #143 closes #133 closes #136 closes #131 closes #81