Re: [heycam/webidl] Import the Promises Guide. (#772)

domenic approved this pull request.

I pushed two commits; please take a look. I don't feel too strongly about either of them, but I do think it'd be nice to avoid spreading "resolve with the void value" too far if possible.

Otherwise, this mostly LGTM. Remaining things are about maybe adding a note around implementations not always creating the PromiseCapability, and maybe avoiding wording that implies you call the author-overwritable `fetch()`. I'll give this approve for now since it's such a nice improvement over the existing ecosystem, so I'd be happy to see it land.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/pull/772#pullrequestreview-279677201