-
Notifications
You must be signed in to change notification settings - Fork 25
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
Support Promise/A+ pattern. #21
Comments
If possible, please also consider dropping pythonic API and clean them up to look more like JS. (check default settings in JSHint/JSLint) |
+1 for supporting promise and making the code conform to conventional javascript style (e.g. camel casing) |
+1 |
@pilwon - when did you fork out? github doesn't show your repo as a tracked fork of |
2 years later... i guess you're not going to support promises? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please support Promise/A+ pattern, especially because
Node.js 0.12
's generators is coming out soon so we'll be able to use it with co or similar.I started this promise-style wrapper project, iron-promise, because the official client doesn't support this yet. I think it's good to put a link to this derivative project so others can find it, but most ideally you will implement this feature in the official library.
The text was updated successfully, but these errors were encountered: