-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add Promise.prototype.finally per tc39 0716 #225
Comments
Makes sense. Feel free to add a PR. |
Any movement here? This one seems alright: |
@xjamundx now I have some problems with time, it will be added when I'll be a little freer. |
I'll integrate this weekend, promise! |
Thanks friends! |
I'd suggest basing it on the proposal's own polyfill at https://github.com/tc39/proposal-promise-finally/blob/master/polyfill.js. It takes one less microtask tick to settle and correctly supports |
PR (#238) created, as promised. Tripped up a bit, perhaps, with the reserved word function name and, possibly, by the species of |
Thanks for bringing it home! |
Since TC39 moved
Promise.prototype.finally
to Stage 2 in the 07 '16 meeting, add tocore.js
. Was requested previously as #70, by others.The text was updated successfully, but these errors were encountered: