-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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
Document promises in promises.markdown. #955
Comments
In my opinion such a page would be really useful. I think it should include information about working with promises in io.js as an ES6 feature, explaining the different issues
I'm still not sure where it should be linked to from or where it belongs since this is a language feature and not a core API - I do think that such a page would be really useful somewhere. |
-1. Not much to say (yet) |
-1, promises are described quite well in the wild |
As long as we only provide docs on node-specific APIs I would think we shouldn't do this. However, I find that this is not a great policy. It's really hard to explain to new users that we document the Buffer object and Url module but not Promise or Math. Maybe it's possible to re-publish (javascript documentation (from mdn) on our website. I guess this is a question for the @iojs/website team. |
I agree with @piscisaureus on this for the same reason. If we include it at all we should be including it as part of a larger ES6 documentation page which would more or less link to various MDN pages. |
-1 on this, promises are not special, nor async, now how to use closures. io.js docs should focus on io.js. |
Good, I think we have a consensus. So closing. @piscisaureus I like your idea - you might want to open a separate issue :) |
When discussing promises in io.js in IRC. @Fishrock123 suggested we consider a promises.markdown file explaining how to use promises in io, explain the rationale and approaches to the rejection event hooks added to
process
and common io.js related issues (working with a promise API and so on).The text was updated successfully, but these errors were encountered: